Skip to content
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](mtmv)fix case of drop index #34959

Merged
merged 2 commits into from
May 17, 2024
Merged

Conversation

zddr
Copy link
Contributor

@zddr zddr commented May 16, 2024

Proposed changes

Issue Number: close #xxx

before drop index , need wait table NORMAL

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@doris-robot
Copy link

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.

@zddr
Copy link
Contributor Author

zddr commented May 16, 2024

run buildall

dataroaring
dataroaring previously approved these changes May 16, 2024
Copy link
Contributor

@dataroaring dataroaring left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

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

@github-actions github-actions bot added approved Indicates a PR has been approved by one committer. reviewed labels May 16, 2024
Copy link
Contributor

PR approved by anyone and no changes requested.

@zddr
Copy link
Contributor Author

zddr commented May 16, 2024

run buildall

@github-actions github-actions bot removed the approved Indicates a PR has been approved by one committer. label May 16, 2024
@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label May 17, 2024
Copy link
Contributor

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

@morrySnow morrySnow merged commit 0f895dd into apache:master May 17, 2024
27 of 29 checks passed
M1saka2003 pushed a commit to M1saka2003/doris that referenced this pull request May 24, 2024
Before dropping the index, we need to wait for the table's status to become NORMAL.
dataroaring pushed a commit that referenced this pull request May 26, 2024
Before dropping the index, we need to wait for the table's status to become NORMAL.
zddr added a commit to zddr/incubator-doris that referenced this pull request May 31, 2024
Before dropping the index, we need to wait for the table's status to become NORMAL.
morrySnow pushed a commit that referenced this pull request May 31, 2024
pick from master #34959

Before dropping the index, we need to wait for the table's status to
become NORMAL.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by one committer. dev/2.1.4-merged reviewed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants