lirui-apache commented on issue #9927: [FLINK-14397][hive] Failed to run Hive UDTF with array arguments URL: https://github.com/apache/flink/pull/9927#issuecomment-544785559 @bowenli86 Thanks for reviewing. > Can we have a separate JIRAfor date type? Sure. > I haven't fully understood what that issue is about and why test passes The issue is Flink by default bridges DATE type to `java.time.LocalDate`, while Hive returns instances of `java.sql.Date` for DATE columns. So there'll be a cast exception if we try to read DATE type from Hive. We have tests of DATE type for metadata but not for read/write real data. That's why the issue hasn't been exposed. > the proposed solution seems not very user friendly In the current solution, users either have to use `HiveTypeUtil.HIVE_DATE_TYPE` or `DataTypes.DATE().bridgeTo(Date.class)`, which seem to be equally friendly/unfriendly to me. Alternatively, we can convert the Date instances returned by Hive to LocalDate instances so that users can continue use `DataTypes.DATE()`. Let's figure that our in the separate task,
---------------------------------------------------------------- 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
