[FLINK-12844] Use default conversion class LocalDate/LocalTime/LocalDateTime for DateType/TimeType/TimestampType in blink

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Component/s: Table SQL / Planner
• Labels: pull-request-available
• Assignee: Jingsong Lee
• Reporter: Jingsong Lee
• Created: 14/Jun/19 06:08
• Updated: 20/Dec/19 08:55
• Resolved: 20/Dec/19 08:55

Description

Now we still use java.sql.Timestamp to be default conversion class for TimestampType.

We should use design of new Type System to use java.time.Local***.

NOTE, This may affect user behaviour:

eg: UDF with eval(Object o), now will pass a java.time.Local*** instead of java.sql.**.

Compatibility method: use DataType.bridgedTo(java.sql.**.class);

Issue Links

• is related to: FLINK-12443 Replace InternalType with LogicalType in blink runner
• relates to: FLINK-12850 Introduce TypeInfo for LocalDate/LocalTime/LocalDateTime
• links to: GitHub Pull Request #8762