ruanwenjun opened a new issue, #10785:
URL: https://github.com/apache/dolphinscheduler/issues/10785

   ### Search before asking
   
   - [X] I had searched in the 
[issues](https://github.com/apache/dolphinscheduler/issues?q=is%3Aissue) and 
found no similar issues.
   
   
   ### What happened
   
   Right now, when we deal the state change event, we will not retry in some 
case, due to we catch the state handle error exception, and don't rethrow it.
   e.g.
   
https://github.com/apache/dolphinscheduler/blob/7d79a2165ee5f0bf1aa7c949c852da179d6c0cc9/dolphinscheduler-master/src/main/java/org/apache/dolphinscheduler/server/master/runner/WorkflowExecuteRunnable.java#L700-L726
   When we update the processInstance failed, we shouldn't catch the exception
   
   And when we update the status to database, we need to recover the status in 
memory, when update to db failed.
   
   ### What you expected to happen
   
   xx
   
   ### How to reproduce
   
   When the database restart, the workflow status will incorrect.
   
   ### Anything else
   
   _No response_
   
   ### Version
   
   dev
   
   ### Are you willing to submit PR?
   
   - [X] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of 
Conduct](https://www.apache.org/foundation/policies/conduct)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: 
[email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to