presto icon indicating copy to clipboard operation
presto copied to clipboard

deprecate joda time : presto-druid

Open Chongliang-Utopia opened this issue 2 years ago • 2 comments

Since Java 8 we have java.time packages which are equivalent to Joda and the recommendation from the author of the Joda-Time is to migrate to java.time(JSR-310) library.

For V9SegmentIndexSource.java, Joda time was not replaced as it instantiates a new instance of SimpleQueryableIndex which takes an Interval (joda.time.Interval) as the argument (line 117), and SimpleQueryableIndex is outside the scope in the external library.

== NO RELEASE NOTE ==

Chongliang-Utopia avatar Jul 15 '22 14:07 Chongliang-Utopia

CLA Signed

The committers listed above are authorized under a signed CLA.

  • :white_check_mark: login: Chongliang-Utopia / name: Chongliang Tao (3fdf3a9b9a9d06db1fdb160cfedd68e615bddaa3)

@Chongliang-Utopia Please squash the commits and use a clean commit message.

ajaygeorge avatar Jul 15 '22 20:07 ajaygeorge