gszadovszky commented on code in PR #466:
URL: https://github.com/apache/parquet-format/pull/466#discussion_r1845468363


##########
LogicalTypes.md:
##########
@@ -609,6 +609,17 @@ that is neither contained by a `LIST`- or `MAP`-annotated 
group nor annotated
 by `LIST` or `MAP` should be interpreted as a required list of required
 elements where the element type is the type of the field.
 
+```

Review Comment:
   Thanks, @etseidl.
   
   Related to protobuf (the parquet-java binding for protobuf) this is the 
other way around currently. There is a flag for converting repeated protobuf 
elements to "3-level" lists but it is false by default, so it converts directly 
to similar repeated elements in the parquet schema ("1-level" list).
   Anyways it seems that neither parquet-rs nor parquet-java (protobuf) should 
have an issue creating proper "3-level" lists. 



-- 
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]

Reply via email to