RussellSpitzer commented on code in PR #10981:
URL: https://github.com/apache/iceberg/pull/10981#discussion_r1767644014
##########
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
|
Review Comment:
might as well fix this one too while we are at 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: 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
