viiccwen opened a new issue, #1184: URL: https://github.com/apache/mahout/issues/1184
ref: #1058 ### What Add direct tests for `qdp/qdp-core/src/readers/parquet.rs`. Current coverage for this file is still zero: - Function coverage: `0.00% (0/31)` - Line coverage: `0.00% (0/415)` ### Why We already have `qdp/qdp-core/tests/parquet_io.rs`, but those tests mainly exercise `io.rs` helper functions. The actual `ParquetReader` and `ParquetStreamingReader` implementations remain uncovered. This is a large and important blind spot in the current Rust coverage report. ### How Add tests that instantiate the reader types directly and exercise both batch and streaming behavior. Suggested coverage targets: - `ParquetReader::new()` rejects missing files and bad schemas - `ParquetReader::read_batch()` handles `List<Float64>` - `ParquetReader::read_batch()` handles `FixedSizeList<Float64>` - reject inconsistent sample sizes across rows - `ParquetStreamingReader::new()` accepts scalar `Float64` for basis encoding - `ParquetStreamingReader` returns chunked data and sample-size metadata correctly - assert empty-file or no-data behavior explicitly This issue is done when `readers/parquet.rs` is no longer at 0% coverage and existing Parquet tests still pass. -- 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]
