-
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](Nereids) offset do more than once when have shuffle after limit (#42576) #42583
Conversation
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
run buildall |
TPC-H: Total hot run time: 49152 ms
|
TPC-DS: Total hot run time: 212209 ms
|
ClickBench: Total hot run time: 30.85 s
|
Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
|
run buildall |
TPC-H: Total hot run time: 49185 ms
|
TPC-DS: Total hot run time: 211754 ms
|
ClickBench: Total hot run time: 30.24 s
|
Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
|
run buildall |
TPC-H: Total hot run time: 49278 ms
|
TPC-DS: Total hot run time: 212199 ms
|
ClickBench: Total hot run time: 30.55 s
|
Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
|
…apache#42576) pick from master apache#42576 intro by apache#39316. it want to fix a problem intro by apache#36699. but forgot to remove all wrong code in apache#36699. after apache#39316, we should not set offset on exchange, when the exchange is on the top of a limit with offset.
run buildall |
TPC-H: Total hot run time: 48685 ms
|
TPC-DS: Total hot run time: 212589 ms
|
ClickBench: Total hot run time: 30.36 s
|
Load test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
|
pick from master #42576
intro by #39316. it want to fix a problem intro by #36699. but forgot to remove all wrong code in #36699.
after #39316, we should not set offset on exchange, when the exchange is on the top of a limit with offset.