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.

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

[Feature][Master] Failover workflow instance in priority #10750

Closed
3 tasks done
zhuxt2015 opened this issue Jul 3, 2022 · 2 comments · Fixed by #10754
Closed
3 tasks done

[Feature][Master] Failover workflow instance in priority #10750

zhuxt2015 opened this issue Jul 3, 2022 · 2 comments · Fixed by #10754
Assignees
@zhuxt2015
Labels
feature new feature

Comments

@zhuxt2015
Copy link
Contributor

Search before asking

  • I had searched in the issues and found no similar feature requirement.

Description

When failing over a workflow instance, it should be based on the priority of the instance

Use case

No response

Related issues

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct
@zhuxt2015 zhuxt2015 added feature new feature Waiting for reply Waiting for reply labels Jul 3, 2022
@github-actions
Copy link

github-actions bot commented Jul 3, 2022

Thank you for your feedback, we have received your issue, Please wait patiently for a reply.

  • In order for us to understand your request as soon as possible, please provide detailed information、version or pictures.
  • If you haven't received a reply for a long time, you can join our slack and send your question to channel #troubleshooting

@Amy0104 Amy0104 removed the Waiting for reply Waiting for reply label Jul 4, 2022
@ruanwenjun
Copy link
Member

ruanwenjun commented Jul 4, 2022 •
edited
Loading

Yes, right now, when we do failover we will not set the process_instance_priority into command, this can be improved.

@zhuxt2015 zhuxt2015 mentioned this issue Jul 4, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@zhuxt2015 zhuxt2015

Labels
feature new feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Feature][Master] Failover workflow instance base on priority zhuxt2015/dolphinscheduler
3 participants
@zhuxt2015 @ruanwenjun @Amy0104