dolphinscheduler icon indicating copy to clipboard operation
dolphinscheduler copied to clipboard

[Bug] [Master] kill task when TaskInstance set TimeoutFailed

Open pegasas opened this issue 1 year ago • 2 comments

Purpose of the pull request

resolve https://github.com/apache/dolphinscheduler/issues/15545

Brief change log

directly kill task if the strategy is timeout_failed.

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)

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

pegasas avatar Feb 01 '24 12:02 pegasas

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (8974233) 38.40% compared to head (9d1ea44) 38.39%.

:exclamation: Current head 9d1ea44 differs from pull request most recent head 56580ab. Consider uploading reports for the commit 56580ab to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##                dev   #15553      +/-   ##
============================================
- Coverage     38.40%   38.39%   -0.02%     
+ Complexity     4753     4749       -4     
============================================
  Files          1305     1305              
  Lines         44841    44837       -4     
  Branches       4800     4799       -1     
============================================
- Hits          17223    17213      -10     
- Misses        25749    25756       +7     
+ Partials       1869     1868       -1     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar Feb 02 '24 02:02 codecov-commenter

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

sonarqubecloud[bot] avatar Feb 02 '24 02:02 sonarqubecloud[bot]