You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Fix: enrichment CSV loader pads missing fields as empty string, not null #383
read_csv_to_batch() in crates/logfwd-io/src/enrichment.rs pads short CSV rows with String::new(), making it impossible to distinguish NULL from empty string in SQL (IS NULL vs = ''). The Arrow schema already marks columns as nullable.
Location
crates/logfwd-io/src/enrichment.rs — search for String::new() in the CSV row padding logic.
Fix
Use Option<String> or build a nullable StringArray where missing fields are null, not empty strings.
Acceptance criteria
Short CSV rows produce null values, not empty strings
IS NULL works correctly in DataFusion queries on enrichment tables
Add a test with short rows verifying null vs empty
Tests pass (just test)
Read AGENTS.md and CONTRIBUTING.md before starting
Activity
fix: remove misplaced doc comment on use statement (CodeRabbit round 1)
google-labs-jules commentedon Mar 31, 2026
Jules has failed to create a task. You can try again later by removing and re-adding the 'jules' label.
google-labs-jules commentedon Mar 31, 2026
Jules is on it. When finished, you will see another comment and be able to review a PR.
google-labs-jules commentedon Mar 31, 2026
Ready for a review! A PR has been created.
fix: enrichment CSV loader pads missing fields with null (#383)