[Bug-8810] [master] add try catch to onSuccess method in the WorkflowExecuteThreadPool

[guoshupei]

This closes #8810

Purpose of the pull request

Brief change log

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:

[caishunfeng]

LGTM

[codecov-commenter]

Codecov Report

Merging #8896 (c99fbea) into dev (c7e80e4) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev    #8896      +/-   ##
============================================
- Coverage     45.29%   45.27%   -0.02%     
+ Complexity     4083     4081       -2     
============================================
  Files           702      702              
  Lines         27186    27186              
  Branches       2908     2908              
============================================
- Hits          12313    12309       -4     
- Misses        13713    13715       +2     
- Partials       1160     1162       +2     

Impacted Files	Coverage Δ
...e/dolphinscheduler/remote/NettyRemotingClient.java	50.70% <0.00%> (-2.82%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c7e80e4...c99fbea. Read the comment docs.

[sonarqubecloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
1 Code Smell

0.0% Coverage
0.0% Duplication