linkedin / ambry

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

Log when a paranoid durability write succeeds. #2822

Closed litingulfs closed 1 month ago

litingulfs commented 1 month ago

To verify that the paranoid durability feature is working as intended this PR adds a log statement for when a request succeeds in ParanoidDurabilityOperationTracker. We are planning to use this log for initial testing and will either change it to debug/trace or remove it in order to prevent flooding logs in later stages of the rollout.

codecov-commenter commented 1 month ago

Codecov Report

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

Project coverage is 70.16%. Comparing base (52ba813) to head (bfad52e). Report is 40 commits behind head on master.

Files Patch % Lines
...bry/router/ParanoidDurabilityOperationTracker.java 60.00% 0 Missing and 2 partials :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2822 +/- ## ============================================ + Coverage 64.24% 70.16% +5.91% - Complexity 10398 11744 +1346 ============================================ Files 840 842 +2 Lines 71755 72356 +601 Branches 8611 8707 +96 ============================================ + Hits 46099 50768 +4669 + Misses 23004 18916 -4088 - Partials 2652 2672 +20 ```

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