scovich commented on code in PR #520:
URL: https://github.com/apache/parquet-format/pull/520#discussion_r2300780297
##########
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.
+When converting an array with (SQL) nullable elements to variant, missing (SQL
NULL) elements must be encoded in `value` as Variant null (`00`).
Review Comment:
Got it. Adjusted the wording a bit to hopefully capture this better.
--
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]
