[ARROW-5967] [Java] DateUtility#timeZoneList is not correct

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.15.0
• Component/s: Java
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/22375
• Assignee: Ji Liu
• Reporter: Ji Liu
• Created: 17/Jul/19 11:36
• Updated: 11/Jan/23 07:43
• Resolved: 26/Jul/19 07:27

Description

Now timeZoneList in DateUtility belongs to Joda time.

Since we have replace Joda time with Java time in ARROW-2015, this should also be changed.

TimeStampXXTZVectors have a timezone member which seems not used now and its getObject returns Long(different with that in TimeStampXXVectors which returns LocalDateTime), should it return LocalDateTime with its timezone?

Is it reasonable if we do as follows:

1. remove Joda timezoneList in DateUtility
2. add method like getLocalDateTimeFromEpochMilli(long epochMillis, String timezone) in DateUtility
3. Not sure make TimeStampXXTZVectors return LocalDateTime？

cc [e-mail]  bryanc  siddteotia

Issue Links

• links to: GitHub Pull Request #4904