emkornfield commented on code in PR #523: URL: https://github.com/apache/parquet-format/pull/523#discussion_r2377681966
########## VariantShredding.md: ########## @@ -38,7 +38,7 @@ All field names of a Variant, whether shredded or not, must be present in the me ## Value Shredding -Variant values are stored in Parquet fields named `value`. +Variant values that are stored in Parquet fields named `value`. Review Comment: I don't think this reads better then existing language. -- 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]
