-
Notifications
You must be signed in to change notification settings - Fork 390
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] Use if function if condition size is 1 #4042
Conversation
Thanks for opening a pull request! Could you open an issue for this pull request on Github Issues? https://github.com/oap-project/gluten/issues Then could you also rename commit message and pull request title in the following format?
See also: |
Run Gluten Clickhouse CI |
Run Gluten Clickhouse CI |
1 similar comment
Run Gluten Clickhouse CI |
3dc3a9d
to
e702a89
Compare
Run Gluten Clickhouse CI |
1 similar comment
Run Gluten Clickhouse CI |
3a3c9b8
to
3ad5a53
Compare
Run Gluten Clickhouse CI |
3ad5a53
to
c53335a
Compare
Run Gluten Clickhouse CI |
c53335a
to
14d738c
Compare
Run Gluten Clickhouse CI |
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: #4041)
If performance is better than mulitiIf, so we use if function if we have only on condition.
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
CI
Function
if
has better performance thanmulitiIf
when there is only one condition, especially after PR:ClickHouse/ClickHouse#57885
On SQL 7954_0
Before:
After PR ClickHouse/ClickHouse#57745
Still if is faster than multiIf: