RussellSpitzer commented on code in PR #10831:
URL: https://github.com/apache/iceberg/pull/10831#discussion_r1826183315
##########
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:
I do agree this should be not-supported for now. Then when shredding is
included say something like for Shredded variants only, binary value
concatenation of meatdata and value + separator byte or something. We can
figure that out with the shredding addition though
--
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]
