GitHub user hakanmemisoglu opened a pull request:
https://github.com/apache/incubator-quickstep/pull/98
QUICKSTEP-53 DateLit improvements
This PR changes DateLit represantation:
- Encode year, month and day information into an u32 field instead of
keeping them separate in three different fields. The change reduces the size of
DateLit from 48 bits to 32 bits.
- With the internal representation change, we changed the comparison
operators' implementation. They now use one unsigned integer comparison for
each method.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/incubator-quickstep date-representation
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/incubator-quickstep/pull/98.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 #98
----
commit 3c8708dfd0987b3491bcabb94e77dd95ea0aea10
Author: Harshad Deshmukh <hbdeshm...@apache.org>
Date: 2016-08-29T19:03:52Z
Separate Date type from Datetime type.
- Beginning from the parser to the operator execution level, treat Date type
sepearately than the Datetime type.
- Provide support for the extract function when applied on the Date
type, implemented in the DateExtractOperation.
- Modified the tests.
commit 998994eee67774d24b4a31a8aea3b1924b0e04ea
Author: Hakan Memisoglu <hakanmemiso...@gmail.com>
Date: 2016-09-02T19:53:47Z
New representation and comparison operators for DateLit.
----
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
