wegamekinglc / calc-flow / 35488990780

93%

Build Type

push

github

Specific Base a71182

Committed by web-flow

Commit Message

test: pin stream_join batching edge cases from review findings (#300)

Copilot review of PR #298 flagged dictionary-column concatenation,
a sequence-overflow edge, and missing byte-budget coverage. Arrow
58.3.0's concat kernel unifies dictionaries or offset-remaps their
keys across batches and errors loudly on dictionary key overflow, so
cross-batch dictionary matches were already correct; a regression
test now pins two distinct dictionaries joined into one output record.

The single-message fast path skipped validate_output_sequence_range
and could build a Batch carrying an exhausted sequence before
failing; it now validates first, pinned by a u64::MAX restore test.
Also adds a byte-axis chunking test with dense per-message sequences
and renames concat_key_column to concat_column since it concatenates
payload columns too.

Co-authored-by: multica-agent <github@multica.ai>

Coverage Stats

191 of 196 new or added lines in 1 file covered. (97.45%)

29 existing lines in 2 files now uncovered.

105295 of 113068 relevant lines covered (93.13%)

103641.32 hits per line