Skip to content
This repository has been archived by the owner on Dec 13, 2023. It is now read-only.

Fix WorkflowRepairService to evaluate the possible task states for Si… #1941

Merged
merged 1 commit into from
Oct 29, 2020

Conversation

kishorebanala
Copy link
Contributor

…mple and system tasks.

@codecov
Copy link

codecov bot commented Oct 29, 2020

Codecov Report

Merging #1941 into dev will increase coverage by 0.03%.
The diff coverage is 85.71%.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev    #1941      +/-   ##
============================================
+ Coverage     65.57%   65.60%   +0.03%     
- Complexity     3928     3936       +8     
============================================
  Files           301      301              
  Lines         18527    18529       +2     
  Branches       1685     1686       +1     
============================================
+ Hits          12149    12156       +7     
+ Misses         5537     5534       -3     
+ Partials        841      839       -2     
Impacted Files Coverage Δ Complexity Δ
...onductor/core/execution/WorkflowRepairService.java 70.27% <85.71%> (+1.69%) 13.00 <9.00> (+5.00)
...e/execution/tasks/SystemTaskWorkerCoordinator.java 83.33% <0.00%> (+2.08%) 18.00% <0.00%> (+1.00%)
.../com/netflix/conductor/dao/mysql/MySQLBaseDAO.java 71.59% <0.00%> (+4.54%) 19.00% <0.00%> (+2.00%)

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 b51ce0a...4fa0b49. Read the comment docs.

@kishorebanala kishorebanala merged commit 6bce391 into dev Oct 29, 2020
@kishorebanala kishorebanala deleted the bugfix/system_tasks_repair branch October 29, 2020 17:07
TwoUnderscorez pushed a commit to TwoUnderscorez/conductor that referenced this pull request Jul 23, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants