lirui-apache commented on issue #10401: [FLINK-15020][hive] Support timestamp type in hive URL: https://github.com/apache/flink/pull/10401#issuecomment-561948816 @xuefuz I just refactored the code such that callers need to explicitly specify whether precision check is needed. Hopefully that'll make things clearer. The conversion from `DataType` to `TypeInfo` is needed in the following code paths: 1. When creating tables via HiveCatalog. As we discussed earlier, we'll enforce the check in this case. 2. When getting `ObjectInspector` for writing to Hive tables. Since Hive tables only support ts(9), we'll enforce the check here. 3. Used by Hive UDFs during initialization, deciding result type, getting `ObjectInspector`s for parameters, etc. We don't enforce the check in this case since we want UDFs to be able to handle ts (<9).
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services
