adamreeve commented on PR #106: URL: https://github.com/apache/parquet-testing/pull/106#issuecomment-4435048763
I tested adapting the C++ tests to use the new 256 bit key files and there was a change needed to account for the definition levels being different to the 128 bit key files, as it looks like they also used the single node list schema. I think this is OK though, we can handle the two different approaches without too much complexity. We didn't notice the same issue with the odd `int64_field.list.int64_field` column path in the existing Rust tests because this field is left unencrypted in the 128 bit key test files that use different encryption keys for columns. -- 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]
