Github user vdiravka commented on a diff in the pull request:
https://github.com/apache/drill/pull/1184#discussion_r181472373
--- Diff: exec/vector/src/main/codegen/templates/FixedValueVectors.java ---
@@ -509,15 +509,15 @@ public long getTwoAsLong(int index) {
public ${friendlyType} getObject(int index) {
org.joda.time.DateTime date = new org.joda.time.DateTime(get(index),
org.joda.time.DateTimeZone.UTC);
date =
date.withZoneRetainFields(org.joda.time.DateTimeZone.getDefault());
- return date;
+ return new java.sql.Date(date.getMillis());
--- End diff --
I think updating to Java8 LocalDate/Time classes would be good choice. And
it will be step forward in the resolving of the Drill's Date/Time issues
mentioned in different Jiras:
[DRILL-5334](https://issues.apache.org/jira/browse/DRILL-5334),
[DRILL-5332](https://issues.apache.org/jira/browse/DRILL-5332) etc.
---
