-
Notifications
You must be signed in to change notification settings - Fork 443
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
[GLUTEN-8168] Add pre-projections for join condition #8185
Conversation
Run Gluten Clickhouse CI on x86 |
1fd58eb
to
31937b3
Compare
Run Gluten Clickhouse CI on x86 |
31937b3
to
5116db8
Compare
Run Gluten Clickhouse CI on x86 |
5116db8
to
44973ca
Compare
Run Gluten Clickhouse CI on x86 |
44973ca
to
11c7e2c
Compare
Run Gluten Clickhouse CI on x86 |
1 similar comment
Run Gluten Clickhouse CI on x86 |
379e848
to
99d6db2
Compare
Run Gluten Clickhouse CI on x86 |
99d6db2
to
bba16e5
Compare
Run Gluten Clickhouse CI on x86 |
bba16e5
to
b966842
Compare
Run Gluten Clickhouse CI on x86 |
b966842
to
17fa836
Compare
Run Gluten Clickhouse CI on x86 |
17fa836
to
d32c1c6
Compare
Run Gluten Clickhouse CI on x86 |
d32c1c6
to
3b8c58a
Compare
Run Gluten Clickhouse CI on x86 |
3b8c58a
to
bea5f6f
Compare
Run Gluten Clickhouse CI on x86 |
bea5f6f
to
82fbf14
Compare
Run Gluten Clickhouse CI on x86 |
...-clickhouse/src/test/scala/org/apache/gluten/execution/GlutenClickHouseTPCHBucketSuite.scala
Outdated
Show resolved
Hide resolved
82fbf14
to
58142af
Compare
Run Gluten Clickhouse CI on x86 |
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.
+1
* refactor * add new rule * fixed
What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
Fixes: #8168
Add pre-projections for left/right plans to eliminate non-attribute expressions in join keys and condition
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
unit tests
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)