GitHub user mehant opened a pull request:
https://github.com/apache/incubator-drill/pull/39
Changes to support date type
DRILL-356
Changes to support Date type
Addressed initial review comments by Jacques
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mehant/incubator-drill DRILL-356
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-drill/pull/39.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 #39
----
commit 06014f05d09d77bb1568e6dfe56888406e36e00f
Author: Mehant Baid <[email protected]>
Date: 2014-03-09T00:53:41Z
Changes to support date type
Rebased on latest master (03/27)
Change tests, cast function rules to work with latest master
Change date type names.
TimeStamp -> TimeStampTZ
DateTime -> TimeStamp
Use 4 bytes for millisecond field in Interval, Interval_Day.
Split the templates into multiple files for readability
Add unit test for sorting on date
Remove generating functions for Nullable type, instead use output type
handling NullIfNull
Use Joda's string parsing to convert from VarChar to date types instead of
our implementation
Remove debug logs and cleanup
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
