Phoenix500526 opened a new pull request, #11090:
URL: https://github.com/apache/arrow-rs/pull/11090

   # Which issue does this PR close?
   
   Part of #9269.
   
   # Rationale for this change
   
   Fixed-file interoperability checks are mixed into the large inline Arrow 
reader test module. Grouping them in the existing fixture module makes related 
tests easier to find.
   
   # What changes are included in this PR?
   
   Move 12 existing Decimal, Float16, BYTE_STREAM_SPLIT, Map, List, and LZ4 
fixture tests into `parquet/tests/arrow_reader/parquet_testing.rs`. Preserve 
their assertions, input files, historical comments, and Miri exclusions, and 
adjust the required imports.
   
   # Are these changes tested?
   
   - `cargo test --locked -p parquet --test arrow_reader parquet_testing::` — 
18 tests passed.
   - `cargo test --locked --offline -p parquet` — passed.
   - `cargo test --locked -p parquet --all-features` — passed.
   - `cargo fmt --all -- --check` — passed.
   
   The 18 fixture tests also pass with default features disabled and 
`arrow,snap,lz4,zstd,flate2-zlib-rs` enabled. Integration runs need the codec 
features used by these fixtures, which the library's unit-test build enables 
through `cfg(test)`.
   
   # Are there any user-facing changes?
   
   Test organization only; no production API or implementation changes.
   


-- 
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]

Reply via email to