GitHub user daveoshinsky opened a pull request:
https://github.com/apache/drill/pull/570
DRILL-4834
decimal implementation is vulnerable to overflow errors, and extremely
complex
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/daveoshinsky/drill DRILL-4834
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/drill/pull/570.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 #570
----
commit 9a47ca52125139d88adf39b5d894a02f870f37d9
Author: U-COMMVAULT-NJ\doshinsky <[email protected]>
Date: 2016-02-09T22:37:47Z
DRILL-4184: support variable length decimal fields in parquet
commit dec00a808c99554f008e23fd21b944b858aa9ae0
Author: daveoshinsky <[email protected]>
Date: 2016-02-09T22:56:28Z
DRILL-4184: changes to support variable length decimal fields in parquet
commit 39bedc5f460fa4ca6158038eb568436a1b0fe3fb
Author: daveoshinsky <[email protected]>
Date: 2016-02-11T15:41:45Z
remove trailing whitespace
commit a41f8d72c538776624d7e1ef79d58ab7e5961e94
Author: daveoshinsky <[email protected]>
Date: 2016-02-11T17:53:04Z
try removing trailing whitespace, again
commit 2a4b57c67bc5e9ad9863e5c1f5693fca9f42e706
Author: Dave Oshinsky <[email protected]>
Date: 2016-03-07T20:56:02Z
centralize decimalLengths access
move declaration to top
commit 68c6c6d72a6bb59362baefa38689f70a02bc0178
Author: Dave Oshinsky <[email protected]>
Date: 2016-03-07T21:52:46Z
change comment
commit 04f7879b44e388c0253b2b641d8ffc0194980cf1
Author: Dave Oshinsky <[email protected]>
Date: 2016-03-07T23:01:38Z
the infamous tab character made a reappearance
commit 838a85747f48e1ba587d2da7d3151d0b1ea295b4
Author: Dave Oshinsky <[email protected]>
Date: 2016-03-16T17:27:52Z
Merge remote-tracking branch 'refs/remotes/apache/master'
commit 4dbf150b17c4a3cc3fc622734f786f575ce87841
Author: Dave Oshinsky <[email protected]>
Date: 2016-06-09T20:28:48Z
Merge remote-tracking branch 'refs/remotes/apache/master'
commit d2372a892216269a71425175c2b2ad9b02941261
Author: Dave Oshinsky <[email protected]>
Date: 2016-08-16T15:27:12Z
Merge remote-tracking branch 'refs/remotes/apache/master'
----
---
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.
---
