sjwiesman commented on a change in pull request #10537: [FLINK-15203][doc] rephrase Hive's data types doc URL: https://github.com/apache/flink/pull/10537#discussion_r356854309
########## File path: docs/dev/table/hive/index.md ########## @@ -283,15 +287,14 @@ Currently `HiveCatalog` supports most Flink data types with the following mappin </tbody> </table> -### Limitations - -The following limitations in Hive's data types impact the mapping between Flink and Hive: -* `CHAR(p)` has a maximum length of 255 -* `VARCHAR(p)` has a maximum length of 65535 +* Hive's `CHAR(p)` has a maximum length of 255 +* Hive's `VARCHAR(p)` has a maximum length of 65535 * Hive's `MAP` only supports primitive key types while Flink's `MAP` can be any data type -* Hive's `UNION` type is not supported -* Flink's `INTERVAL` type cannot be mapped to Hive `INTERVAL` type -* Flink's `TIMESTAMP_WITH_TIME_ZONE` and `TIMESTAMP_WITH_LOCAL_TIME_ZONE` are not supported by Hive -* Flink's `TIMESTAMP_WITHOUT_TIME_ZONE` type cannot be mapped to Hive's `TIMESTAMP` type due to precision difference. -* Flink's `MULTISET` is not supported by Hive + + +Note that: + +* Flink doesn't support Hive's `UNION` type is not supported +* Hive doesn't support Flink's `TIMESTAMP_WITH_TIME_ZONE`, `TIMESTAMP_WITH_LOCAL_TIME_ZONE`, and `MULTISET` +* Flink's `INTERVAL` type cannot be mapped to Hive `INTERVAL` type yet Review comment: ```suggestion * Flink's `INTERVAL` type cannot be mapped to Hive `INTERVAL` type ``` ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
