Skip to content

Conversation

@darach
Copy link
Owner

@darach darach commented Jan 26, 2026

Signed-off-by: Darach Ennis darach@gmail.com

Description

Motivation and Context

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update
  • Performance improvement
  • Refactoring (no functional changes)
  • CI/CD changes
  • Dependencies update

Checklist

  • I have read the CONTRIBUTING guide
  • My code follows the project's style guidelines
  • I have run cargo fmt and cargo clippy
  • I have added tests that prove my fix/feature works
  • All new and existing tests pass (cargo test)
  • I have updated the documentation accordingly
  • My changes generate no new warnings

Testing

Performance Impact

Additional Notes

Signed-off-by: Darach Ennis <darach@gmail.com>
@coveralls
Copy link

Pull Request Test Coverage Report for Build 21372980149

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.002%) to 79.789%

Totals Coverage Status
Change from base Build 21372874911: 0.002%
Covered Lines: 34788
Relevant Lines: 43600

💛 - Coveralls

@darach darach merged commit 6e58519 into main Jan 26, 2026
27 of 29 checks passed
@github-actions
Copy link

✅ Benchmark Results: No significant regressions

Click to expand benchmark comparison
group                                          main                                   pr
-----                                          ----                                   --
diff_comparison/compute_diff/array_change      1.00    722.3±5.41ns        ? ?/sec    1.22   880.9±29.98ns        ? ?/sec
diff_comparison/roundtrip/array_change         1.00  1500.1±15.22ns        ? ?/sec    1.25  1880.6±66.56ns        ? ?/sec
diff_comparison/roundtrip/field_add            1.00   777.0±12.27ns        ? ?/sec    1.16   903.8±12.58ns        ? ?/sec
diff_comparison/roundtrip/field_remove         1.00   785.9±10.94ns        ? ?/sec    1.18   928.9±10.38ns        ? ?/sec
diff_comparison/roundtrip/small_change         1.00  1026.4±23.80ns        ? ?/sec    1.21  1239.5±16.90ns        ? ?/sec
diff_patch_merge/crdt_lww/single_comparison    1.00      0.6±0.01ns        ? ?/sec    2.49      1.6±0.00ns        ? ?/sec
format/ison/parsing/structural/large           1.00     26.3±0.87µs   653.1 MB/sec    1.15     30.4±0.51µs   565.5 MB/sec
format/latency/toon/tiny                       1.00     67.3±3.62ns        ? ?/sec    1.16     78.0±3.07ns        ? ?/sec
format/toon/parsing/structural/large           1.00      9.7±1.08µs   507.8 MB/sec    1.21     11.7±0.04µs   421.1 MB/sec
format/toon/parsing/structural/small           1.26    765.4±9.64ns   297.8 MB/sec    1.00   605.6±10.14ns   376.4 MB/sec
streaming/allocation/fresh_processor           1.20    414.1±6.31µs   278.5 MB/sec    1.00    346.2±7.42µs   333.2 MB/sec
streaming/allocation/reused_processor          1.18    412.6±4.73µs   279.6 MB/sec    1.00   349.1±13.36µs   330.5 MB/sec
streaming/chunk_throughput/chunk_size/1000     1.23      4.3±0.02ms   279.3 MB/sec    1.00      3.5±0.04ms   344.4 MB/sec
streaming/chunk_throughput/chunk_size/2000     1.19      4.2±0.02ms   287.0 MB/sec    1.00      3.5±0.02ms   342.2 MB/sec
streaming/chunk_throughput/chunk_size/500      1.20      4.3±0.03ms   280.1 MB/sec    1.00      3.5±0.02ms   336.4 MB/sec
streaming/jsonl_batch_size/optimized/10000     1.00      3.3±0.03ms   356.0 MB/sec    1.25      4.2±0.02ms   285.3 MB/sec
streaming/simd_comparison/simd_batch/1000      1.19    412.1±3.83µs   279.9 MB/sec    1.00    345.3±7.16µs   334.0 MB/sec

Threshold: 15% regression triggers warning


Benchmarks compare this PR against main branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants