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

Reply via email to