alamb commented on PR #7687: URL: https://github.com/apache/arrow-rs/pull/7687#issuecomment-3070267620
@emkornfield are you opposed to merging this PR? I can't quite tell from the comments > I think general consensus is we should update the spec (I think there is a PR open for this). I did make a PR here to clarify the spec about Int96 stats (with a recommendation, not with any actual change): - https://github.com/apache/parquet-format/pull/504 > Beyond that there was discussion on whether we should have a new sort order or just rely on versions. I'll start a thread but #7909 is pertinent to a discussion on on how much effort adding a SortOrder would be. In my opinion, there is no reason to hold up this PR (which improves compatibility for an "implementation defined" part of the spec) on actually changing the spec. We can proceed with actually defining a SortOrder / fixing the spec in parallel -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
