[GitHub] [flink] sjwiesman commented on a change in pull request #10537: [FLINK-15203][doc] rephrase Hive's data types doc

[GitBox <gi...@apache.org>]

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_r356854131
 
 

 ##########
 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
 
 Review comment:
   ```suggestion
   * Flink doesn't support Hive's `UNION` 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:
users@infra.apache.org


With regards,
Apache Git Services