scovich commented on code in PR #520:
URL: https://github.com/apache/parquet-format/pull/520#discussion_r2300771549
##########
VariantShredding.md:
##########
@@ -142,26 +152,26 @@ optional group tags (VARIANT) {
}
```
-All elements of an array must be present (not missing) because the `array`
Variant encoding does not allow missing elements.
-That is, either `typed_value` or `value` (but not both) must be non-null.
-Null elements must be encoded in `value` as Variant null: basic type 0
(primitive) and physical type 0 (null).
+All elements of a variant array must be present (not missing) because the
`array` Variant encoding does not allow missing elements.
+That is, either `typed_value` or `value` (but not both) must be present.
Review Comment:
Should that be a separate issue/PR since it would be a "behavior change" of
sorts?
Or did you want me to adjust the wording here?
--
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]
