alamb opened a new pull request, #504: URL: https://github.com/apache/parquet-format/pull/504
### Rationale for this change - As can be seen on https://github.com/apache/parquet-format/issues/502 and the linked issues, the current state of Int96 is confusing. - Given there are proposed changes to various open source parquet writers to make them work better with Int96 despite it being deprecated, we should clarify the spec with respect to Int96 and statistics ### What changes are included in this PR? 1. Add comments clarifying that Int96 should not be written by new parquet writers (@rdblue's suggestion of what deprecated means in practice) 2. Add a recommendation on ordering of Int96 statistics to match what @rahulketch and @alkis are suggesting on https://github.com/apache/parquet-java/issues/3242 and https://github.com/apache/arrow-rs/issues/7686) ### Do these changes have PoC implementations? No -- in my mind this does not change the meaning or intent of the parquet spec, but instead adds clarification to help other implementer -- 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]
