Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue 3223: RetentionTest failure #3303

Merged
merged 1 commit into from
Jan 22, 2019
Merged

Conversation

RaulGracia
Copy link
Contributor

Change log description
Cherry pick changes from PR #3272 to branch r0.4.

Purpose of the change
Fixes #3223.

What the code does
For a description, please check PR #3272.

How to verify it
For a description, please check PR #3272.

Signed-off-by: Raúl Gracia <raul.gracia@emc.com>
@codecov
Copy link

codecov bot commented Jan 21, 2019

Codecov Report

Merging #3303 into r0.4 will decrease coverage by 0.07%.
The diff coverage is 78.94%.

Impacted file tree graph

@@             Coverage Diff              @@
##               r0.4    #3303      +/-   ##
============================================
- Coverage     78.09%   78.02%   -0.08%     
+ Complexity     7305     7303       -2     
============================================
  Files           571      571              
  Lines         29241    29259      +18     
  Branches       2694     2696       +2     
============================================
- Hits          22837    22829       -8     
- Misses         4829     4845      +16     
- Partials       1575     1585      +10
Impacted Files Coverage Δ Complexity Δ
...ller/store/stream/AbstractStreamMetadataStore.java 92% <100%> (+0.03%) 117 <1> (+1) ⬆️
...ller/store/stream/InMemoryStreamMetadataStore.java 79.08% <100%> (+0.13%) 49 <1> (+1) ⬆️
.../controller/server/retention/StreamCutService.java 79.03% <100%> (+0.34%) 15 <2> (+1) ⬆️
...controller/store/stream/ZKStreamMetadataStore.java 84.95% <73.33%> (-0.83%) 73 <6> (+6)
...main/java/io/pravega/controller/task/TaskBase.java 77.57% <0%> (-6.55%) 26% <0%> (-3%)
...ntstore/server/host/ZKSegmentContainerManager.java 87.5% <0%> (-6.25%) 6% <0%> (ø)
...erver/logs/operations/CheckpointOperationBase.java 90.9% <0%> (-4.55%) 4% <0%> (-1%)
...egmentstore/server/containers/MetadataCleaner.java 90.56% <0%> (-3.78%) 14% <0%> (-2%)
...o/pravega/segmentstore/server/logs/DurableLog.java 87.8% <0%> (-1.96%) 66% <0%> (-2%)
.../server/logs/SegmentMetadataUpdateTransaction.java 88.39% <0%> (-1.79%) 82% <0%> (-2%)
... and 9 more

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 775bc48...5025c5d. Read the comment docs.

@fpj fpj merged commit 27d0545 into pravega:r0.4 Jan 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants