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

fix for system task with external payload storage #1365

Merged
merged 1 commit into from Oct 24, 2019

Conversation

apanicker-nflx
Copy link
Collaborator

No description provided.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 3255

  • 14 of 14 (100.0%) changed or added relevant lines in 3 files are covered.
  • 1 unchanged line in 1 file lost coverage.
  • Overall coverage increased (+0.04%) to 70.218%

Files with Coverage Reduction New Missed Lines %
server/src/main/java/com/netflix/conductor/server/JerseyModule.java 1 90.91%
Totals Coverage Status
Change from base Build 3251: 0.04%
Covered Lines: 10275
Relevant Lines: 14633

💛 - Coveralls

@codecov
Copy link

codecov bot commented Oct 24, 2019

Codecov Report

Merging #1365 into dev will increase coverage by 0.04%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##                dev    #1365      +/-   ##
============================================
+ Coverage     64.31%   64.36%   +0.04%     
- Complexity     2847     2850       +3     
============================================
  Files           241      241              
  Lines         14316    14324       +8     
  Branches       1409     1410       +1     
============================================
+ Hits           9207     9219      +12     
+ Misses         4329     4325       -4     
  Partials        780      780
Impacted Files Coverage Δ Complexity Δ
...flix/conductor/core/execution/WorkflowSweeper.java 75% <100%> (ø) 8 <1> (ø) ⬇️
...lix/conductor/core/execution/WorkflowExecutor.java 76.78% <100%> (+0.17%) 138 <0> (+1) ⬆️
...tflix/conductor/core/execution/DeciderService.java 87.73% <100%> (+0.11%) 109 <1> (+1) ⬆️
...ductor/core/utils/ExternalPayloadStorageUtils.java 77.77% <0%> (+4.93%) 14% <0%> (+1%) ⬆️

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 3f66857...63c9407. Read the comment docs.

@apanicker-nflx apanicker-nflx merged commit 1b5ca23 into dev Oct 24, 2019
@apanicker-nflx apanicker-nflx deleted the system_task_external_payload branch October 24, 2019 16:49
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.

None yet

2 participants