[BEAM-10611] Use ZetaSQL Value.createDatetimeValue(LocalDateTime) function to avoid extra conversions

• Type: Improvement
• Status: Resolved
• Priority: P3
• Resolution: Fixed
• Fix Version/s: 2.24.0
• Component/s: dsl-sql-zetasql
• Assignee: Robin Qiu
• Reporter: Robin Qiu
• Created: 30/Jul/20 21:37
• Updated: 15/May/21 17:56
• Resolved: 15/May/21 17:56

Description

GoogleSQL (internal) has a Value.createDatetimeValue() method that takes a Java LocalDateTime, but it has not been open-sourced to ZetaSQL. We should open source this function and then Beam should use it to replace the current DATETIME type translation implementation. It will avoid unnecessary conversions and make the code simpler.

Issue Links

• blocks: BEAM-10238 [Umbrella] Improve Beam ZetaSQL type system
• links to: GitHub Pull Request #12536