[fix](Nereids) offset do more than once when have shuffle after limit #42576

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.

Sign up for GitHub

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

Jump to bottom

Merged

morrySnow merged 1 commit into apache:master from morrySnow:fix_limit

Oct 28, 2024

Contributor

morrySnow commented Oct 28, 2024

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.

doris-robot commented Oct 28, 2024

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

morrySnow added p0_w dev/2.0.x dev/2.1.x dev/3.0.x labels

morrySnow added a commit to morrySnow/incubator-doris that referenced this pull request


          [fix](Nereids) offset do more than once when have shuffle after limit (…

474084a

…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.


          [fix](Nereids) offset do more than once when have shuffle after limit

b782c46

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.

morrySnow force-pushed the fix_limit branch from c45ee1e to b782c46 Compare

October 28, 2024 07:22

morrySnow added a commit to morrySnow/incubator-doris that referenced this pull request


          [fix](Nereids) offset do more than once when have shuffle after limit (…

caa0933

…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.

Contributor Author

morrySnow commented Oct 28, 2024

run buildall

morrySnow mentioned this pull request

[fix](Nereids) offset do more than once when have shuffle after limit (#42576) #42577

Merged

morrySnow added a commit to morrySnow/incubator-doris that referenced this pull request


          [fix](Nereids) offset do more than once when have shuffle after limit (…

0c6afcd

…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.

morrySnow added a commit to morrySnow/incubator-doris that referenced this pull request


          [fix](Nereids) offset do more than once when have shuffle after limit (…

42b1970

…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.

morrySnow mentioned this pull request

[fix](Nereids) offset do more than once when have shuffle after limit (#42576) #42583

Merged

924060929 approved these changes

View reviewed changes

Contributor

github-actions bot commented Oct 28, 2024

PR approved by at least one committer and no changes requested.

github-actions bot added the approved label

Contributor

github-actions bot commented Oct 28, 2024

PR approved by anyone and no changes requested.

github-actions bot added the reviewed label

starocean999 approved these changes

View reviewed changes

morrySnow merged commit 45d0e01 into apache:master

morrySnow deleted the fix_limit branch

October 28, 2024 09:39

github-actions bot added the cherry-pick-conflict-in-3.0 label

morrySnow added a commit to morrySnow/incubator-doris that referenced this pull request


          [fix](Nereids) offset do more than once when have shuffle after limit (…

469b8e8

…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.

morrySnow removed the cherry-pick-conflict-in-3.0 label

morrySnow added a commit that referenced this pull request


          [fix](Nereids) offset do more than once when have shuffle after limit (…

0baca54

…#42576) (#42577)

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.

morrySnow added dev/2.1.7-merged and removed dev/2.1.x labels

morrySnow mentioned this pull request

[fix](Nereids) offset do more than once when have shuffle after limit (#42576) #42682

Merged

morrySnow added a commit that referenced this pull request


          [fix](Nereids) offset do more than once when have shuffle after limit (…

3dd0b03

…#42576) (#42682)

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.

morrySnow added dev/3.0.3-merged and removed dev/3.0.x labels

morrySnow added a commit to morrySnow/incubator-doris that referenced this pull request


          [fix](Nereids) offset do more than once when have shuffle after limit (…

2569ed4

…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.

morrySnow added a commit that referenced this pull request


          [fix](Nereids) offset do more than once when have shuffle after limit (…

cbb97f8

…#42576) (#42583)

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.

morrySnow added dev/2.0.16-merged and removed dev/2.0.x labels

gavinchou mentioned this pull request

Release Note 3.0.3 #44522

Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved dev/2.0.16-merged dev/2.1.7-merged dev/3.0.3-merged p0_w reviewed