rok commented on code in PR #7111: URL: https://github.com/apache/arrow-rs/pull/7111#discussion_r2007606880
########## parquet/tests/arrow_reader/encryption.rs: ########## @@ -201,3 +211,410 @@ fn verify_encryption_test_file_read(file: File, decryption_properties: FileDecry verify_encryption_test_data(record_batches, metadata); } + +fn row_group_sizes(metadata: &ParquetMetaData) -> Vec<i64> { + metadata.row_groups().iter().map(|x| x.num_rows()).collect() +} + +#[test] +fn test_uniform_encryption_roundtrip() { Review Comment: Done: https://github.com/apache/arrow-rs/pull/7111/commits/ce8d2a91e2163791f88ac9286c62cb1a41ad2c86 Not sure if this is the best way to split tests, but it does remove a lot of `#cfg` logic. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org