[Bug] [Master] When worker down, some taskInstance has been dispatched may not be failovered.

[ruanwenjun]

Search before asking

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

What happened

I find some taskInstance may not be failovered when the worker down, and the taskInstance's status is DISPATCH.

When we do worker failover, we need to find out which taskInstances belong to the need failover worker, we use taskInstance's host to do this judge, if the taskInstance's host == needFailoveredWorker, then we may failover this task(We have extra judgement).

The problem is that when we success dispatch the taskInstance to a worker, the taskIntance's host may still null, if then the worker down, this taskInstance may not be failovered.

What you expected to happen

The worker failover can work will.

How to reproduce

Start some master and worker, and submit a lot of processInstance, then kill one worker.

Anything else

No response

Version

dev

Are you willing to submit PR?

• Yes I am willing to submit a PR!

Code of Conduct

• I agree to follow this project's Code of Conduct

[github-actions]

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