gszadovszky commented on code in PR #466:
URL: https://github.com/apache/parquet-format/pull/466#discussion_r1841869882
##########
LogicalTypes.md:
##########
@@ -670,6 +681,8 @@ optional group array_of_arrays (LIST) {
#### Backward-compatibility rules
Review Comment:
The current spec (and it didn't change with this PR) specifies the "3-level"
structure and the "unannotated" (aka. "1-level") list types as ones to be
supported. So, according to the spec, a writer either use the first or the
second one but cannot use both in the same schema.
Since only the "3-level" structure is capable of properly describing
nullability at any level, this should be the preferred one.
We may start a discussion to deprecate the "1-level" list but I think this
would be a different topic. This PR is about clarifying the backward
compatibility rules only.
--
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]
