Skip to content

Conversation

@zddr
Copy link
Contributor

@zddr zddr commented Sep 19, 2024

  • When deleting a job, do not query and delete it first. Instead, call the method to delete the job directly. If the job does not exist(When the materialized view only creates the table and the job is not yet created, the materialized view is concurrently deleted), throw an exception
  • When changing mtmv, narrow down the scope of the lock and place it in each sub method

@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 Sep 19, 2024

run buildall

1 similar comment
@zddr
Copy link
Contributor Author

zddr commented Sep 19, 2024

run buildall

@github-actions github-actions bot added the approved Indicates a PR has been approved by one committer. label Sep 19, 2024
@github-actions
Copy link
Contributor

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

@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@zddr
Copy link
Contributor Author

zddr commented Sep 23, 2024

run buildall

@zddr
Copy link
Contributor Author

zddr commented Sep 23, 2024

run p0

@zddr
Copy link
Contributor Author

zddr commented Sep 24, 2024

run cloud_p0

@morrySnow morrySnow merged commit 185eda3 into apache:master Sep 25, 2024
zddr added a commit to zddr/incubator-doris that referenced this pull request Sep 25, 2024
- When deleting a job, do not query and delete it first. Instead, call
the method to delete the job directly. If the job does not exist(When
the materialized view only creates the table and the job is not yet
created, the materialized view is concurrently deleted), throw an
exception
- When changing mtmv, narrow down the scope of the lock and place it in
each sub method
zddr added a commit to zddr/incubator-doris that referenced this pull request Sep 25, 2024
- When deleting a job, do not query and delete it first. Instead, call
the method to delete the job directly. If the job does not exist(When
the materialized view only creates the table and the job is not yet
created, the materialized view is concurrently deleted), throw an
exception
- When changing mtmv, narrow down the scope of the lock and place it in
each sub method
@morrySnow morrySnow added the p0_b label Sep 25, 2024
dataroaring pushed a commit that referenced this pull request Sep 26, 2024
- When deleting a job, do not query and delete it first. Instead, call
the method to delete the job directly. If the job does not exist(When
the materialized view only creates the table and the job is not yet
created, the materialized view is concurrently deleted), throw an
exception
- When changing mtmv, narrow down the scope of the lock and place it in
each sub method
@yiguolei yiguolei mentioned this pull request Nov 6, 2024
@yiguolei yiguolei mentioned this pull request Jan 19, 2025
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.7-merged dev/3.0.2-merged p0_b reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants