autoCAT6 opened a new pull request, #1438:
URL: https://github.com/apache/arrow-go/pull/1438

   ### Rationale for this change
   
   The encryption configuration suite omits two encrypted columns with 
AES_GCM_CTR_V1 and a plaintext footer. The shared C++ fixture added in 
apache/parquet-testing#130 covers this combination and was generated using a 
branch containing apache/arrow#54884.
   
   This follows the writer correction in #1412 with a reusable interoperability 
regression. Adam Reeve authored the shared fixture.
   
   ### What changes are included in this PR?
   
   * Add the missing writer configuration for DataPageV1/V2 and 
Snappy/uncompressed output.
   * Read generated Go files and the shared C++ fixture through the existing 
key-retrieval, explicit-key, AAD-mismatch and plaintext-only checks, including 
values and page indexes.
   * Fast-forward the parquet-testing submodule from 92d45b0 to eedfad3, which 
includes the fixture and intervening dataset updates. No production code 
changes.
   
   ### Are these changes tested?
   
   * The focused suite passes with `-race -tags assert` and with `-tags 
assert,noasm`.
   * Temporarily restoring the old #1412 metadata logic makes all four new 
generated-file variants fail. The shared C++ fixture's four decryption 
configurations still pass in that control. The production source was restored 
afterward.
   * All 13 test-bearing Parquet packages pass with the race detector; five 
additional packages have no test files.
   * Vet and repository-pinned lint pass for `./parquet`; gofmt, diff checks 
and Apache RAT pass.
   
   ### Are there any user-facing changes?
   
   No API or runtime behavior changes.
   
   AI was used for the tests and PR description.
   


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