scovich commented on code in PR #520: URL: https://github.com/apache/parquet-format/pull/520#discussion_r2300821583
########## VariantShredding.md: ########## @@ -110,18 +119,19 @@ Shredded values must use the following Parquet types: Primitive values can be shredded using the equivalent Parquet primitive type from the table above for `typed_value`. -Unless the value is shredded as an object (see [Objects](#objects)), `typed_value` or `value` (but not both) must be non-null. +Unless the value is shredded as an object (see [Objects](#objects)), exactly one of `typed_value` or `value` must be present in each row. +Missing values in an optional group are encoded as missing values of the entire group. Review Comment: I tried rewording a bit, see what you think? (If we still need a full schema we could do that) -- 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]
