Skip to content

Fix mmap reader yielding zero rows when the first row exceeds the head buffer - #338

Merged
vincentlaucsb merged 2 commits into
masterfrom
claude/csv-parser-issue-337-864c98
Oct 10, 2026
Merged

vincentlaucsb merged 2 commits into
masterfrom
claude/csv-parser-issue-337-864c98

Only read past the mmap head when it holds no complete row

f890eca
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Oct 10, 2026 in 1s

91.51% (-0.04%) compared to eee1732

View this Pull Request on Codecov

91.51% (-0.04%) compared to eee1732

Details

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.51%. Comparing base (eee1732) to head (f890eca).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #338      +/-   ##
==========================================
- Coverage   91.54%   91.51%   -0.04%     
==========================================
  Files          55       55              
  Lines        3595     3605      +10     
  Branches     1068     1072       +4     
==========================================
+ Hits         3291     3299       +8     
- Misses        115      116       +1     
- Partials      189      190       +1     

☔ 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.