-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[fix](nereids)support group_concat with distinct and order by #38080
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
Conversation
|
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
fcfd2f8 to
d2f9b65
Compare
|
run buildall |
TPC-H: Total hot run time: 41914 ms |
TPC-DS: Total hot run time: 168737 ms |
ClickBench: Total hot run time: 29.81 s |
d2f9b65 to
9fa7a8c
Compare
|
run buildall |
TPC-H: Total hot run time: 41761 ms |
|
run buildall |
TPC-H: Total hot run time: 41436 ms |
TPC-DS: Total hot run time: 170687 ms |
ClickBench: Total hot run time: 29.86 s |
d7dcee5 to
60d2df9
Compare
|
run buildall |
TPC-H: Total hot run time: 41780 ms |
TPC-DS: Total hot run time: 168853 ms |
ClickBench: Total hot run time: 29.62 s |
60d2df9 to
ea90c5a
Compare
|
run buildall |
|
clang-tidy review says "All clean, LGTM! 👍" |
TPC-H: Total hot run time: 41720 ms |
TPC-DS: Total hot run time: 168254 ms |
ClickBench: Total hot run time: 29.97 s |
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
## Proposed changes pick from master #38080 <!--Describe your changes.-->
This pr support group_concat with distinct and order by. So we can support sql like bellow: ``` select group_concat(distinct x order by y) from t; select multi_distinct_group_concat(x order by y) from t; ```
This pr support group_concat with distinct and order by. So we can support sql like bellow: ``` select group_concat(distinct x order by y) from t; select multi_distinct_group_concat(x order by y) from t; ```
Proposed changes
This pr support group_concat with distinct and order by. So we can support sql like bellow: