Fix Presto's format_datetime function with time zone #11283
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary:
The format_datetime currently outptus the time zone id if for 3 or fewer 'z' characters
in the format string. However, the JODA library, which this is based on, does this for
3 or more 'Z' characters.
https://www.joda.org/joda-time/apidocs/org/joda/time/format/DateTimeFormat.html
This diff fixes this, as well as adds support for a single 'Z' (which outputs the same
thing as 'ZZ' just without the colon). So 'Z' is fully supported for any number of
characters.
Differential Revision: D64500193