docete opened a new pull request #10877: [FLINK-15602][table-planner-blink] Padding TIMESTAMP type to respect … URL: https://github.com/apache/flink/pull/10877 …the precision when casting timestamp to varchar ## What is the purpose of the change According to SQL 2011 Part 2 Section 6.13 General Rules 11) d) > If SD is a datetime data type or an interval data type then let Y be the shortest character string that > conforms to the definition of <literal> in Subclause 5.3, “<literal>”, and such that the interpreted value of Y is SV and the interpreted precision of Y is the precision of SD. This PR padding the TIMESTAMP type to respect the precision when casting timestamp to varchar. ## Brief change log - c53d150 Padding TIMESTAMP type to respect precision ## Verifying this change This change is already covered by existing tests, such as *TempralTypeTests*. ## Does this pull request potentially affect one of the following parts: - Dependencies (does it add or upgrade a dependency): (yes / **no**) - The public API, i.e., is any changed class annotated with `@Public(Evolving)`: (yes / **no**) - The serializers: (yes / **no** / don't know) - The runtime per-record code paths (performance sensitive): (yes / **no** / don't know) - Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (yes / **no** / don't know) - The S3 file system connector: (yes / **no** / don't know) ## Documentation - Does this pull request introduce a new feature? (**yes** / no) - If yes, how is the feature documented? (not applicable / docs / **JavaDocs** / not documented)
---------------------------------------------------------------- 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
