GitHub user jiang-wu opened a pull request:
https://github.com/apache/drill/pull/1184
DRILL-6242 - Use java.sql.[Date|Time|Timestamp] classes to hold valueâ¦
See Jira ticket for details. Use java.sql.Date, java.sql.Time, and
java.sql.Timestamp as the Java representation for their corresponding Drill
types. This does not lose any precisions as these classes are just simple
subclasses of java.util.Date with millisecond precision. But using these
classes allows the command line to properly format the data using
org.apache.drill.exec.util.JsonStringArrayList and
org.apache.drill.exec.util.JsonStringHashMap.
The changes are simple enough. But many Test** methods need to be updated
to use java.sql.Date|Time|Timestamp. Opt not to optimize these changes.
Places still use joda DateTime to parse date and time as before, but then
converted to the java.sql.Date|Time|Timestamp as appropriate.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jiang-wu/drill DRILL-6242-master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/drill/pull/1184.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1184
----
commit 7cbb8b81196732cb223c031cd629d9bc941640d9
Author: Jiang Wu <jiang.wu@...>
Date: 2018-03-22T20:42:20Z
DRILL-6242 - Use java.sql.[Date|Time|Timestamp] classes to hold values from
corresponding Drill date, time, and timestamp types.
----
---
