[FLINK-21456] TableResult#print() should correctly stringify values

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 1.15.0
• Component/s: Table SQL / API
• Labels: auto-deprioritized-major auto-unassigned pull-request-available
• Assignee: Francesco Guardiani
• Reporter: Jark Wu
• Created: 23/Feb/21 09:35
• Updated: 30/Nov/21 16:17
• Resolved: 30/Nov/21 16:17

Description

Currently TableResult#print()} simply use {{Object#toString() as the string representation of the fields. This is not SQL compliant, because for TIMESTAMP and TIMESTAMP_LZ, the string representation should be 2021-02-23 17:30:00 instead of 2021-02-23T17:30:00Z.

Note: we may need to update PrintUtils#rowToString(Row) and also SQL Client which invokes this method.

Issue Links

• is related to: FLINK-24579 FlinkSQL does not print correctly content of arrays nested into maps
• relates to: FLINK-18550 use TableResult#collect to get select result for sql client
• relates to: FLINK-21454 FLIP-163: SQL Client Improvements
• relates to: FLINK-24403 Improve behaviour of CAST
• links to: GitHub Pull Request #17405

Sub-Tasks

• TableResult#print() should use internal data types: Closed
• Move SqlDateTimeUtils to table-common: Closed
• Refactor casting rules in a similar fashion to DataStructureConverter: Closed
• Port to string casting rules to the new CastRule interface: Closed
• Use the new casting rules in TableResult#print: Closed
• Cleanup all the various date/time/timestamp utils: Closed