rzhang10 commented on code in PR #4301:
URL: https://github.com/apache/iceberg/pull/4301#discussion_r859151207
##########
format/spec.md:
##########
@@ -1128,15 +1128,15 @@ This serialization scheme is for storing single values
as individual binary valu
| **`long`** | **`JSON long`** | `34`
| |
| **`float`** | **`JSON number`** | `1.0`
| |
| **`double`** | **`JSON number`** | `1.0`
| |
-| **`decimal(P,S)`** | **`JSON number`** | `14.20`
| Stores the decimal value as a general json
number |
+| **`decimal(P,S)`** | **`JSON number`** | `14.20`
| Stores the decimal as a number with S places
after the decimal |
| **`date`** | **`JSON string`** |
`"2017-11-16"` | Stores ISO-8601 standard date |
| **`time`** | **`JSON string`** |
`"22:31:08.123456"` | Stores ISO-8601 standard time with
microsecond precision |
| **`timestamp`** | **`JSON string`** |
`"2017-11-16T22:31:08.123456"` | Stores ISO-8601 standard timestamp
with microsecond precision; must not include a zone offset |
| **`timestamptz`** | **`JSON string`** |
`"2017-11-16T22:31:08.123456-07:00"` | Stores ISO-8601 standard timestamp
with microsecond precision; must include a zone offset |
-| **`string`** | **`JSON string`** | `"foo"`
| |
-| **`uuid`** | **`JSON string`** |
`"eb26bdb1-a1d8-4aa6-990e-da940875492c"` | Stores the lowercase uuid string |
-| **`fixed(L)`** | **`JSON string`** | `"0x3162"`
| Stored as a hexadecimal string, prefixed by
`0x` |
-| **`binary`** | **`JSON string`** | `"0x3162"`
| Stored as a hexadecimal string, prefixed by
`0x` |
+| **`string`** | **`JSON string`** | `"iceberg"`
| |
+| **`uuid`** | **`JSON string`** |
`"f79c3e09-677c-4bbd-a479-3f349cb785e7"` | Stores the lowercase uuid string |
+| **`fixed(L)`** | **`JSON string`** |
`"0x00010203"` | Stored as a hexadecimal string,
prefixed by `0x` |
Review Comment:
Should we also note that the hex string length should be less than or equal
to 2*L
--
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]
