szehon-ho commented on code in PR #10981:
URL: https://github.com/apache/iceberg/pull/10981#discussion_r1882776271
##########
format/spec.md:
##########
@@ -1480,6 +1497,9 @@ This serialization scheme is for storing single values as
individual binary valu
| **`struct`** | Not supported
|
| **`list`** | Not supported
|
| **`map`** | Not supported
|
+| **`geometry`** | A single point, encoded as a {x, y, optional
z, optional m} concatenation of its 8-byte little-endian IEEE 754 coordinate
values, with the optional coordinates encoded as NaN if unset. |
+| **`geography`** | A single point, encoded as a {x, y, optional
z, optional m} concatenation of its 8-byte little-endian IEEE 754 coordinate
values, with the optional coordinates encoded as NaN if unset. |
Review Comment:
Yea this was @rdblue suggestion, because the lower_bound and upper_bound is
a point, then we can make it more smaller than WKB, which is signfiicant
because this is on every manifest entry. The comment is here
https://github.com/apache/iceberg/pull/10981#discussion_r1768947092 , we had
another discussion but cant find it.
--
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]
