Uploaded image for project: 'Apache Arrow'
  1. Apache Arrow
  2. ARROW-5967

[Java] DateUtility#timeZoneList is not correct

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.15.0
    • Component/s: Java

      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 Micah Kornfield  Bryan Cutler  Siddharth Teotia

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                tianchen92 Ji Liu
                Reporter:
                tianchen92 Ji Liu
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 1.5h
                  1.5h