jackye1995 commented on code in PR #9717: URL: https://github.com/apache/iceberg/pull/9717#discussion_r1495041325
########## open-api/rest-catalog-open-api.yaml: ########## @@ -3324,6 +3348,217 @@ components: type: integer format: int64 + BooleanTypeValue: + type: boolean + example: true + + IntegerTypeValue: + type: integer + example: 42 + + LongTypeValue: + type: integer + format: int64 + example: 9223372036854775807 + + FloatTypeValue: + type: number + format: float + example: 3.14 + + DoubleTypeValue: + type: number + format: double + example: 123.456 + + DecimalTypeValue: + type: string + description: + "Decimal type values are serialized as strings. Decimals with a positive scale serialize as numeric plain + text, while decimals with a negative scale use scientific notation and the exponent will be equal to + the negated scale" + example: + positiveScale: "123.4500" + zeroScale: "2" + negativeScale: "2E+20" + + StringTypeValue: + type: string + example: "hello" + + UUIDTypeValue: + type: string + format: uuid + description: + "UUID type values are serialized as a lowercase string" + example: "eb26bdb1-a1d8-4aa6-990e-da940875492c" + + DateTypeValue: + type: string + format: date + description: + "Date type values follow the `YYYY-MM-DD` ISO-8601 standard date format" + example: "2007-12-03" + + TimeTypeValue: + type: string + description: + "Time type values follow the `HH:MM[:SS[.sssssssss]]` ISO-8601 format with microsecond precision" + example: "22:31:08.123456" + + TimestampTypeValue: + type: string + description: + "Timestamp type values follow the 'YYYY-MM-DDTHH:MM[:SS[.sssssssss]][+00:00]' ISO-8601 format with microsecond + precision. Timestamps that adjust to UTC include a `+00:00` offset. Without this offset, the format implies + local time" + example: + withoutTimezone: "2007-12-03T10:15:30" + withTimezone: "2007-12-03T10:15:30+00:00" + withFractionalSeconds: "2007-03-25T12:34:56.123456" + + FixedTypeValue: + type: string + description: + "Fixed length type values are stored and serialized as a hexadecimal string preserving the fixed length" + example: "000102ff" + + BinaryTypeValue: + type: string + description: + "Binary type values are stored and serialized as a hexadecimal string" + example: "000102ff" + + MapTypeValue: + type: object + description: + "A map structure serialized with keys and values arrays that maintain type" + properties: + keys: + type: array + items: + $ref: '#/components/schemas/TypeValue' + values: + type: array + items: + $ref: '#/components/schemas/TypeValue' + example: + { + keys: [ 1, 2 ], Review Comment: This spec is currently derived based on the `SingleValueParser` logic that's why it is generic. But if we want it to to focused on just content file serialization, I think we should have dedicated schemas like `ColumnSizesMap`, `ColumnCountsMap`, `ColumnBoundsMap`, etc. What do you think? -- 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