Ensuring that AadMetadata and EncryptionAlgorithm initialization is deterministic

[argmarco-tkd]

Ensuring that AadMetadata and EncryptionAlgorithm initialization is deterministic

We found that GitHub Actions for MinGW complained with an error similar to

Expected: decryptor_.DecryptFile(file_path, dec_props) doesn't throw an exception.
  Actual: it throws parquet::ParquetException with description "Couldn't deserialize thrift: TProtocolException: Invalid data
".

The root cause of this error was tracked down to the fields of AadMetadata (parquet/types.h) and EncryptionAlgorithm and (parquet/metadata.cc) not being consistently initialized (essentially, they were left to being initialized to whatever the platform/OS/environment decided to).

In this change, we ensure that the data structures are initialized in a deterministic fashion.

Tests

• Existing tests pass (via ctest -L parquet)
• Error in CI (GitHub Actions) is no longer present
  • Before: https://github.com/protegrity/arrow/actions/runs/22082358998/job/63810006516
  • After: https://github.com/protegrity/arrow/actions/runs/22119243127/job/63935097395

[github-actions]

Thanks for opening a pull request!

If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose

Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename the pull request title in the following format?

GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

See also:

• Other pull requests
• Contribution Guidelines - Contributing Overview

[sofia-tekdatum]

Thanks!