rdblue commented on code in PR #12658:
URL: https://github.com/apache/iceberg/pull/12658#discussion_r2017535903


##########
format/spec.md:
##########
@@ -1558,6 +1561,7 @@ The binary single-value serialization can be used to 
store the lower and upper b
 
|------------------------------|-------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
 | **`geometry`**               | A single point, encoded as a x:y:z:m 
concatenation of its 8-byte little-endian IEEE 754 coordinate values. x and y 
are mandatory. This becomes x:y if z and m are both unset, x:y:z if only m is 
unset, and x:y:NaN:m if only z is unset. |
 | **`geography`**              | A single point, encoded as a x:y:z:m 
concatenation of its 8-byte little-endian IEEE 754 coordinate values. x and y 
are mandatory. This becomes x:y if z and m are both unset, x:y:z if only m is 
unset, and x:y:NaN:m if only z is unset. |
+| **`variant`**              | A `Variant` object, where each subcolumn path 
serves as a key and the corresponding bound value as the value. Subcolumn paths 
follow the JSON path format. |

Review Comment:
   `Variant` is not something that we can reference here because that is a 
class in the implementation. I think what you want to say is that the 
serialized value is a variant metadata (v1) concatenated with a variant object. 
The object's fields are field paths in the normalized JSON path format and the 
values are the upper or lower bound corresponding to the shredded type in the 
data file.



-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org

Reply via email to