[FLINK-23073] Fix space handling in Row CSV timestamp parser

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.14.0, 1.13.2
• Fix Version/s: 1.14.0, 1.13.2
• Component/s: Formats (JSON, Avro, Parquet, ORC, SequenceFile)
• Labels: pull-request-available
• Assignee: Seth Wiesman
• Reporter: Seth Wiesman
• Created: 21/Jun/21 18:36
• Updated: 23/Sep/21 17:27
• Resolved: 23/Jun/21 15:08

Description

FLINK-21947 Added support for TIMESTAMP_LTZ in the CSV format by replacing java.sql.Timestamp.valueOf with java.time.LocalDateTime.parse. Timestamp.valueOf internally calls `trim()` on the string before parsing while LocalDateTime.parse does not. This caused a breaking change where the CSV format can no longer parse timestamps of CSV's with spaces after the delimiter. We should manually re-add the call to trim to revert the behavior.

Issue Links

• links to: GitHub Pull Request #16228