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

fix case where queue wait time can be negative #1426

Merged
merged 1 commit into from Dec 4, 2019

Conversation

apanicker-nflx
Copy link
Collaborator

No description provided.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 3450

  • 4 of 4 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.006%) to 70.086%

Totals Coverage Status
Change from base Build 3439: 0.006%
Covered Lines: 11138
Relevant Lines: 15892

💛 - Coveralls

@codecov
Copy link

codecov bot commented Nov 30, 2019

Codecov Report

Merging #1426 into dev will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##               dev    #1426      +/-   ##
===========================================
+ Coverage     64.3%   64.31%   +<.01%     
- Complexity    3189     3192       +3     
===========================================
  Files          262      262              
  Lines        15580    15583       +3     
  Branches      1521     1523       +2     
===========================================
+ Hits         10019    10022       +3     
  Misses        4723     4723              
  Partials       838      838
Impacted Files Coverage Δ Complexity Δ
.../netflix/conductor/common/metadata/tasks/Task.java 81.33% <100%> (+0.25%) 93 <0> (+3) ⬆️

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 db4eb41...81ba8ea. Read the comment docs.

@apanicker-nflx apanicker-nflx merged commit 40cfad0 into dev Dec 4, 2019
@apanicker-nflx apanicker-nflx deleted the queue_wait_time_compute_fix branch December 4, 2019 01:15
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

3 participants