-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[CH-186] support RangePartitioning
#189
[CH-186] support RangePartitioning
#189
Conversation
Can one of the admins verify this patch? |
rely on apache/incubator-gluten#524 |
a54acfe
to
bdd81c3
Compare
test this please with 524 |
1 similar comment
test this please with 524 |
4b17432
to
9abb5fc
Compare
test this please with 524 |
6b29620
to
41dce3a
Compare
test this please with 524 |
test this please with 524 |
As comment, there is a optimization to do:
|
4168d74
to
c97a6ef
Compare
test this please with 524 |
1 similar comment
test this please with 524 |
7fc45d7
to
ba4f6fe
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
see issue #186
We implement
RangePartitionNativeSplitter
in this pr which support running range partitioning in native backend.This pr is mainly for improving the
order by
clause.Some benchmark test by running
order by
.