fix: Make read_csv_batched
respect skip_rows
and skip_lines
#23484
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes issue 23483.
Fixes issue where the
skip_lines
parameter inread_csv_batched
was silently ignored due to being incorrectly mapped towith_skip_rows
in the Rust layer. Users could not skip lines using this argument as intended, diverging behavior fromread_csv
.skip_lines
towith_skip_lines
in the PyBatchedCsv constructor.skip_lines
andskip_rows
to matchread_csv