autoCAT6 opened a new pull request, #3925: URL: https://github.com/apache/parquet-java/pull/3925
### Rationale for this change The encryption matrix omits AES_GCM_CTR_V1 with a plaintext footer. Cover this combination using Adam Reeve's shared C++ sample from apache/parquet-testing#130, alongside the Java writer's own round trip. ### What changes are included in this PR? * Add the configuration to the existing writer/decryption and interoperability matrices, including projection of plaintext columns without decryption properties. * Pin the new sample to eedfad3f086678d282a00d8ff3cf8932d5d29af5. Keep the six existing samples at their current revision. * Check the complete row count in both read paths. The shared samples contain 50 rows; adjust the expected data accordingly. ### Are these changes tested? On JDK 17, macOS ARM64: * `./mvnw -pl parquet-hadoop verify -Dit.test=ITTestEncryptionOptions -Dmaven.javadoc.skip=true -Dsource.skip=true`: 882 unit cases passed, 26 skipped; the encryption integration test passed. * A valid zero-row replacement for the new shared file is ignored by the original suite. The extended suite fails with expected 50 / actual 0. Restoring the real sample passes. * Spotless and RAT checks pass. The zero-row control was local validation tooling and is not included in this patch. ### Are there any user-facing changes? Test coverage only. AI-generated test 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
