-
Notifications
You must be signed in to change notification settings - Fork 370
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-5668][CH] Support mixed conditions in shuffle hash join #5735
Conversation
Run Gluten Clickhouse CI |
54df6f0
to
c4e0058
Compare
Run Gluten Clickhouse CI |
c4e0058
to
a81db7e
Compare
Run Gluten Clickhouse CI |
a81db7e
to
24194a0
Compare
Run Gluten Clickhouse CI |
24194a0
to
83480f7
Compare
Run Gluten Clickhouse CI |
83480f7
to
57f5ec0
Compare
Run Gluten Clickhouse CI |
57f5ec0
to
341264a
Compare
Run Gluten Clickhouse CI |
341264a
to
e42389a
Compare
Run Gluten Clickhouse CI |
e42389a
to
6d96819
Compare
Run Gluten Clickhouse CI |
6d96819
to
111d5c3
Compare
Run Gluten Clickhouse CI |
111d5c3
to
ba84a5e
Compare
Run Gluten Clickhouse CI |
ba84a5e
to
ab20403
Compare
Run Gluten Clickhouse CI |
ab20403
to
3a9d8bb
Compare
Run Gluten Clickhouse CI |
3a9d8bb
to
a4ef682
Compare
Run Gluten Clickhouse CI |
a4ef682
to
f77b039
Compare
Run Gluten Clickhouse CI |
1 similar comment
Run Gluten Clickhouse CI |
60295b2
to
2d663e7
Compare
Run Gluten Clickhouse CI |
因为 BHJ 右表构建的逻辑改变了,测试结果如下:
受
|
Run Gluten Clickhouse CI |
Gluten Perf Test Pass, GlutenWithCHStandard tpch-data-sf100, mean-total 118264, min-total 115868. |
分析了下
疑问在于 补充,追踪 |
4ff4a33
to
48b4bd3
Compare
Run Gluten Clickhouse CI |
48b4bd3
to
b5004f7
Compare
Run Gluten Clickhouse CI |
b5004f7
to
977347e
Compare
Run Gluten Clickhouse CI |
Run Gluten Clickhouse CI |
977347e
to
92f7b06
Compare
Run Gluten Clickhouse CI |
92f7b06
to
3404b1a
Compare
Run Gluten Clickhouse CI |
3404b1a
to
a125b16
Compare
Run Gluten Clickhouse CI |
a125b16
to
da5361f
Compare
Run Gluten Clickhouse CI |
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
What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
Fixes: #5668
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)