RussellSpitzer commented on code in PR #10981:
URL: https://github.com/apache/iceberg/pull/10981#discussion_r1767643715
##########
format/spec.md:
##########
@@ -1312,7 +1335,7 @@ This serialization scheme is for storing single values as
individual binary valu
| **`struct`** | **`JSON object by field ID`** | `{"1": 1,
"2": "bar"}` | Stores struct fields using the field ID as
the JSON field name; field values are stored using this JSON single-value
format |
| **`list`** | **`JSON array of values`** | `[1, 2, 3]`
| Stores a JSON array of values that are
serialized using this JSON single-value format |
| **`map`** | **`JSON object of key and value arrays`** | `{ "keys":
["a", "b"], "values": [1, 2] }` | Stores arrays of keys and values; individual
keys and values are serialized using this JSON single-value format |
-
+| **`geometry`** | **`JSON string`**
|`00000000013FF00000000000003FF0000000000000`| Stores WKB as a hexideciamal
string, see Appendix G |
Review Comment:
link again
##########
format/spec.md:
##########
@@ -1265,27 +1287,28 @@ Example
This serialization scheme is for storing single values as individual binary
values in the lower and upper bounds maps of manifest files.
-| Type | Binary serialization
|
-|------------------------------|--------------------------------------------------------------------------------------------------------------|
-| **`boolean`** | `0x00` for false, non-zero byte for true
|
-| **`int`** | Stored as 4-byte little-endian
|
-| **`long`** | Stored as 8-byte little-endian
|
-| **`float`** | Stored as 4-byte little-endian
|
-| **`double`** | Stored as 8-byte little-endian
|
-| **`date`** | Stores days from the 1970-01-01 in an 4-byte
little-endian int |
-| **`time`** | Stores microseconds from midnight in an
8-byte little-endian long |
-| **`timestamp`** | Stores microseconds from 1970-01-01
00:00:00.000000 in an 8-byte little-endian long |
-| **`timestamptz`** | Stores microseconds from 1970-01-01
00:00:00.000000 UTC in an 8-byte little-endian long |
-| **`timestamp_ns`** | Stores nanoseconds from 1970-01-01
00:00:00.000000000 in an 8-byte little-endian long |
-| **`timestamptz_ns`** | Stores nanoseconds from 1970-01-01
00:00:00.000000000 UTC in an 8-byte little-endian long |
-| **`string`** | UTF-8 bytes (without length)
|
-| **`uuid`** | 16-byte big-endian value, see example in
Appendix B |
-| **`fixed(L)`** | Binary value
|
-| **`binary`** | Binary value (without length)
|
-| **`decimal(P, S)`** | Stores unscaled value as two’s-complement
big-endian binary, using the minimum number of bytes for the value |
-| **`struct`** | Not supported
|
-| **`list`** | Not supported
|
-| **`map`** | Not supported
|
+| Type | Binary serialization
|
+|----------------------|--------------------------------------------------------------------------------------------------------------|
+| **`boolean`** | `0x00` for false, non-zero byte for true
|
+| **`int`** | Stored as 4-byte little-endian
|
+| **`long`** | Stored as 8-byte little-endian
|
+| **`float`** | Stored as 4-byte little-endian
|
+| **`double`** | Stored as 8-byte little-endian
|
+| **`date`** | Stores days from the 1970-01-01 in an 4-byte
little-endian int |
+| **`time`** | Stores microseconds from midnight in an 8-byte
little-endian long |
+| **`timestamp`** | Stores microseconds from 1970-01-01 00:00:00.000000
in an 8-byte little-endian long |
+| **`timestamptz`** | Stores microseconds from 1970-01-01 00:00:00.000000
UTC in an 8-byte little-endian long |
+| **`timestamp_ns`** | Stores nanoseconds from 1970-01-01 00:00:00.000000000
in an 8-byte little-endian long |
+| **`timestamptz_ns`** | Stores nanoseconds from 1970-01-01 00:00:00.000000000
UTC in an 8-byte little-endian long |
+| **`string`** | UTF-8 bytes (without length)
|
+| **`uuid`** | 16-byte big-endian value, see example in Appendix B
|
+| **`fixed(L)`** | Binary value
|
+| **`binary`** | Binary value (without length)
|
+| **`decimal(P, S)`** | Stores unscaled value as two’s-complement big-endian
binary, using the minimum number of bytes for the value |
+| **`struct`** | Not supported
|
+| **`list`** | Not supported
|
+| **`map`** | Not supported
|
+| **`geometry`** | WKB format, see Appendix G
|
Review Comment:
link
--
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]
