-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
[fix](function)adjust aggregate function's nullable property #37330
Conversation
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 39888 ms
|
TPC-DS: Total hot run time: 173170 ms
|
ClickBench: Total hot run time: 30.86 s
|
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 39935 ms
|
TPC-DS: Total hot run time: 173261 ms
|
ClickBench: Total hot run time: 30.49 s
|
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 40189 ms
|
TPC-DS: Total hot run time: 171730 ms
|
ClickBench: Total hot run time: 30.45 s
|
clang-tidy review says "All clean, LGTM! 👍" |
run buildall |
TPC-H: Total hot run time: 40306 ms
|
TPC-DS: Total hot run time: 174645 ms
|
ClickBench: Total hot run time: 30.31 s
|
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 39921 ms
|
TPC-DS: Total hot run time: 173601 ms
|
ClickBench: Total hot run time: 31 s
|
run buildall |
clang-tidy review says "All clean, LGTM! 👍" |
run buildall |
TPC-H: Total hot run time: 39922 ms
|
TPC-DS: Total hot run time: 174565 ms
|
ClickBench: Total hot run time: 31.01 s
|
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
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
…37330) ## Proposed changes in this PR have change some age function nullable property. aggregate functions can be divided into 3 types by their nullable properties: - AlwaysNullable - AlwaysNotNullable - NullableAggregateFunction this PR code of FE part from apache#37215, the author is @starocean999 --------- Co-authored-by: starocean999 <[email protected]>
## Proposed changes in this PR have change some age function nullable property. aggregate functions can be divided into 3 types by their nullable properties: - AlwaysNullable - AlwaysNotNullable - NullableAggregateFunction this PR code of FE part from #37215, the author is @starocean999 --------- Co-authored-by: starocean999 <[email protected]>
## Proposed changes Issue Number: close #xxx before `corr(nullable_x, nullable_y)` will core dump. not fixed. no need to patch in master because the refactor #37330 already changed the implementation context
Proposed changes
in this PR have change some age function nullable property.
aggregate functions can be divided into 3 types by their nullable properties:
this PR code of FE part from #37215, the author is @starocean999