search

linkedin / ambry

Distributed object store
https://github.com/linkedin/ambry/wiki
Apache License 2.0
1.74k stars 275 forks source link

[Paranoid Durability] Unit Tests and Bugfix #2803

Closed litingulfs closed 2 months ago

litingulfs commented 2 months ago

This PR fixes a bug in ParanoidDurabilityOperationTracker where we didn't complete initialization before checking whether enough good replicas are present. It also adds unit tests.

codecov-commenter commented 2 months ago

Codecov Report

Attention: Patch coverage is 77.77778% with 2 lines in your changes missing coverage. Please review.

Project coverage is 70.12%. Comparing base (52ba813) to head (72f9a2f). Report is 27 commits behind head on master.

Files Patch % Lines
...bry/router/ParanoidDurabilityOperationTracker.java 77.77% 1 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2803 +/- ## ============================================ + Coverage 64.24% 70.12% +5.87% - Complexity 10398 11718 +1320 ============================================ Files 840 842 +2 Lines 71755 72241 +486 Branches 8611 8696 +85 ============================================ + Hits 46099 50657 +4558 + Misses 23004 18917 -4087 - Partials 2652 2667 +15 ```

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