Skip to content

Expose read-only consumer batches and clarify Channel comparisons - #66

Merged
eventhorizon-cli merged 3 commits into
mainfrom
feat/read-only-consumer-batches
Oct 24, 2026
Merged

eventhorizon-cli merged 3 commits into
mainfrom
feat/read-only-consumer-batches

Conversation

@eventhorizon-cli

Copy link
Copy Markdown
Owner

Consumers now receive IReadOnlyList<T> batches, exposing Count and indexed access without an extra container or copy. Memory and MemoryMappedFile batches remain valid after later pulls, commits, and segment recycling; pull/push contracts, samples, tests, and bilingual package documentation are updated together.

Compatibility: this changes public interface signatures. Applications and libraries must be recompiled; custom implementations, mocks, and push handlers must update batch types.

The Channel comparison adds measured application-side array batching, with all 20 elapsed-time/allocation results and reproduction details retained. README elapsed-time charts keep native per-item Channel reads, while allocation charts compare retained batches. Auto/Manual allocation measurements are nearly equal and share one explicitly labelled representative curve. Bilingual design diagrams and benchmark charts use a consistent palette and typography.

Validation:

  • Format verification, restore, Release build, and all 890 test executions passed across .NET 8 and .NET 10 (308 Core and 137 MMF tests per framework).
  • All 113 existing benchmark cases were rerun successfully; 20 additional Channel array-batching cases were measured. Existing results are retained with their original configuration and measured runtime version.
  • Repeated concurrency checks and retained-batch adapter checks passed. SVG rendering, visual inspection, local image links, and whitespace checks passed.

@codecov

codecov Bot commented Oct 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.94%. Comparing base (96ed057) to head (9db9e20).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #66   +/-   ##
=======================================
  Coverage   92.94%   92.94%           
=======================================
  Files          34       34           
  Lines        1955     1955           
  Branches      356      356           
=======================================
  Hits         1817     1817           
  Misses         86       86           
  Partials       52       52           
Flag Coverage Δ
unit-tests 92.94% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@eventhorizon-cli
eventhorizon-cli merged commit c276547 into main Oct 24, 2026
5 checks passed
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.

1 participant