[DSIP-82][Master/Worker] Use FAILOVER_FINISH_NODES to avoid duplicate workflow/task when failover - Githubissues

apache / dolphinscheduler

Apache DolphinScheduler is the modern data orchestration platform. Agile to create high performance workflow with low-code

https://dolphinscheduler.apache.org/

Apache License 2.0

12.89k stars 4.63k forks source link

[DSIP-82][Master/Worker] Use FAILOVER_FINISH_NODES to avoid duplicate workflow/task when failover #16821

Closed ruanwenjun closed 2 days ago

ruanwenjun commented 3 days ago

Purpose of the pull reques

related issue #16759 close #16825

Brief change log

Remove waiting strategy
Add delay time to failover event and check the server health to avoid failover reconnect server
Use FailoverFinishNodes to avoid one node has been failovered but it reconnect to registry.

Verify this pull request

This pull request is code cleanup without any test coverage.

(or)

This pull request is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(or)

Pull Request Notice

Pull Request Notice

If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

sonarcloud[bot] commented 2 days ago

Quality Gate failed

Failed conditions
27.4% Coverage on New Code (required ≥ 60%)

See analysis details on SonarQube Cloud