aquinoonell opened a new pull request, #11977:
URL: https://github.com/apache/arrow-rs/pull/11977
> [!IMPORTANT]
>
> CI is enabled on my fork.
# Which issue does this PR close?
- Part of #9348
# Rationale for this change
#11063 moved the public-API async reader tests into one integration file.
#9348 was reopened so those tests can be split into topical modules, following
`parquet/tests/arrow_reader/row_filter/`.
This PR establishes `parquet/tests/arrow_reader/async_reader/` and moves the
offset-index tests. The remaining groups stay in `async_reader/mod.rs` for
follow-up PRs.
# What changes are included in this PR?
- Rename `parquet/tests/arrow_reader/async_reader.rs` to
`parquet/tests/arrow_reader/async_reader/mod.rs`.
- Move these tests into `async_reader/offset_index.rs`:
- `non_empty_offset_index_doesnt_panic_in_read_row_group`
- `empty_offset_index_doesnt_panic_in_column_chunks`
- `test_cached_array_reader_sparse_offset_error`
No production code, assertions, or public APIs change. Tests that use
private helpers stay in `parquet/src/arrow/async_reader/mod.rs`.
# Are these changes tested?
`cargo test -p parquet --features async --test arrow_reader -- offset_index`
# Are there any user-facing changes?
No.
# AI usage disclosure
This pull request was prepared with an AI assistant in Cursor.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
