MaxGekk opened a new pull request #27617: [SPARK-30865][SQL] Refactor 
DateTimeUtils
URL: https://github.com/apache/spark/pull/27617
 
 
   ### What changes were proposed in this pull request?
   
   1. Move TimeZoneUTC and TimeZoneGMT to DateTimeTestUtils
   2. Remove TimeZoneGMT
   3. Use ZoneId.systemDefault() instead of defaultTimeZone().toZoneId
   4. Alias SQLDate & SQLTimestamp to internal types of DateType and 
TimestampType
   
   ### Why are the changes needed?
   1. TimeZoneUTC and TimeZoneGMT are moved to DateTimeTestUtils because they 
are used only in tests
   2. TimeZoneGMT can be removed because it is equal to TimeZoneUTC
   3. After the PR #27494, Spark expressions and DateTimeUtils functions 
switched to ZoneId instead of TimeZone completely. `defaultTimeZone()` with 
`TimeZone` as return type is not needed anymore.
   4. SQLDate and SQLTimestamp types can be explicitly aliased to internal 
types of DateType and and TimestampType instead of declaring this in a comment.
   
   ### Does this PR introduce any user-facing change?
   No
   
   ### How was this patch tested?
   By existing test suites
   

----------------------------------------------------------------
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:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to