JFinis opened a new pull request, #491: URL: https://github.com/apache/parquet-format/pull/491
This PR contains the small documentation fixes that were pulled out of [PR#221](https://github.com/apache/parquet-format/pull/221), [as it was suggested](https://github.com/apache/parquet-format/pull/221#discussion_r1451853168) that they should go into an own PR. Mostly, this is small format and and content fixes. The only larger change is that the explanation of comparison rules was deduplicated out of `LogicalTypes.md`, as the same text is also in `parquet.thrift` and keeping such duplicate documentation in sync is hard. Instead, `LogicalTypes.md` just points to `parquet.thrift`. Deduplicating this is especially important for PR#221, as that PR introduces a new column ordering and thus makes the ordering rules even larger, so having them duplicated is becoming even a bigger burden. -- 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]
