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

[ARCTIC-512][AMS] if table is running, add failed tasks to execute #513

Merged
merged 2 commits into from
Oct 25, 2022

Conversation

hzluting
Copy link
Contributor

Why are the changes needed?

fix #512

Brief change log

  • if table is running, add failed tasks to execute

How was this patch tested?

  • Add some test cases that check the changes thoroughly including negative and positive cases if possible

  • Add screenshots for manual tests if appropriate

  • Run test locally before making a pull request

Documentation

  • Does this pull request introduces a new feature? (yes / no)
  • If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@wangtaohz
Copy link
Contributor

LGTM

@wangtaohz wangtaohz merged commit 9fa2572 into apache:master Oct 25, 2022
hzluting added a commit to hzluting/arctic that referenced this pull request Oct 25, 2022
…pache#513)

* fix apache#512 if table is running, add failed tasks to execute

Co-authored-by: luting <dylzlt93299@gmail.com>
@hzluting hzluting deleted the fix-failed-retry branch October 25, 2022 08:39
hzluting added a commit to hzluting/arctic that referenced this pull request Oct 25, 2022
…pache#513)

* fix apache#512 if table is running, add failed tasks to execute

Co-authored-by: luting <dylzlt93299@gmail.com>
wangtaohz pushed a commit that referenced this pull request Oct 25, 2022
)

[ARCTIC-512][AMS] if table is running, add failed tasks to execute (#513)

* fix #512 if table is running, add failed tasks to execute

Co-authored-by: luting <dylzlt93299@gmail.com>

Co-authored-by: luting <dylzlt93299@gmail.com>
zhoujinsong pushed a commit that referenced this pull request May 31, 2023
)

* fix #512 if table is running, add failed tasks to execute

Co-authored-by: luting <dylzlt93299@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug][AMS]: Failed optimize task can't retry
2 participants