Fokko commented on code in PR #7917: URL: https://github.com/apache/iceberg/pull/7917#discussion_r1243193249
########## docs/spark-writes.md: ########## @@ -397,25 +397,26 @@ so you may want to understand the type conversion in Iceberg in prior to design This type conversion table describes how Spark types are converted to the Iceberg types. The conversion applies on both creating Iceberg table and writing to Iceberg table via Spark. -| Spark | Iceberg | Notes | -|-----------------|-------------------------|-------| -| boolean | boolean | | -| short | integer | | -| byte | integer | | -| integer | integer | | -| long | long | | -| float | float | | -| double | double | | -| date | date | | -| timestamp | timestamp with timezone | | -| char | string | | -| varchar | string | | -| string | string | | -| binary | binary | | -| decimal | decimal | | -| struct | struct | | -| array | list | | -| map | map | | +| Spark | Iceberg | Notes | +|-----------------|----------------------------|-------| +| boolean | boolean | | +| short | integer | | +| byte | integer | | +| integer | integer | | +| long | long | | +| float | float | | +| double | double | | +| date | date | | +| timestamp | timestamp with timezone | | +| timestampNtz | timestamp without timezone | | Review Comment: ```suggestion | timestamp_ntz | timestamp without timezone | | ``` ########## docs/spark-writes.md: ########## @@ -438,7 +439,7 @@ This type conversion table describes how Iceberg types are converted to the Spar | date | date | | | time | | Not supported | | timestamp with timezone | timestamp | | -| timestamp without timezone | | Not supported | +| timestamp without timezone | timestampNtz | | Review Comment: ```suggestion | timestamp without timezone | timestamp_ntz | | ``` -- 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]
