aihuaxu commented on code in PR #10831:
URL: https://github.com/apache/iceberg/pull/10831#discussion_r1854522786
##########
format/spec.md:
##########
@@ -1436,6 +1457,7 @@ This serialization scheme is for storing single values as
individual binary valu
| **`struct`** | Not supported
|
| **`list`** | Not supported
|
| **`map`** | Not supported
|
+| **`variant`** | Not supported
|
Review Comment:
Seems binary representation is used for lower bound and upper bound and JSON
single-value serialization is used for default value. Looks like they are
defined independently? But since there is no default value for Variant, i will
remove from JSON section.
--
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]