wjones127 commented on code in PR #35146:
URL: https://github.com/apache/arrow/pull/35146#discussion_r1191710190
##########
cpp/src/parquet/properties.h:
##########
@@ -938,13 +937,13 @@ class PARQUET_EXPORT ArrowWriterProperties {
/// list types (default "item"), will use "entries", as is specified in
Review Comment:
You are right, that should be "element".
Parquet will always have them as "element". Arrow has it's own spec, but
isn't prescriptive for the name of the list type. Either way, I don't think we
should care too much about this detail; these field names should be pretty much
invisible to users.
--
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]
