prasanthj commented on a change in pull request #1392:
URL: https://github.com/apache/hive/pull/1392#discussion_r468862678
##########
File path:
common/src/java/org/apache/hadoop/hive/common/type/TimestampTZUtil.java
##########
@@ -145,13 +149,33 @@ public static ZoneId parseTimeZone(String timeZoneStr) {
}
}
+ public static Timestamp convertTimestampToZone(Timestamp ts, ZoneId
fromZone, ZoneId toZone) {
+ return convertTimestampToZone(ts, fromZone, toZone, false);
+ }
+
/**
* Timestamps are technically time zone agnostic, and this method sort of
cheats its logic.
* Timestamps are supposed to represent nanos since [UTC epoch]. Here,
* the input timestamp represents nanoseconds since [epoch at fromZone], and
* we return a Timestamp representing nanoseconds since [epoch at toZone].
*/
- public static Timestamp convertTimestampToZone(Timestamp ts, ZoneId
fromZone, ZoneId toZone) {
+ public static Timestamp convertTimestampToZone(Timestamp ts, ZoneId
fromZone, ZoneId toZone,
+ boolean legacyConversion) {
+ if (legacyConversion) {
+ try {
+ DateFormat formatter = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
Review comment:
should this be a threadlocal value? using this way is pretty slow iirc..
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
