search

linkedin / ambry

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

Fix missing data chunk metric again #2875

Closed justinlin-linkedin closed 2 months ago

justinlin-linkedin commented 2 months ago

Summary

In https://github.com/linkedin/ambry/pull/2867, we move the missing data chunk metric to the right place, but forgot to remove it from the processGetBlobResponse function. This PR fixes it.

Test

unit test

codecov-commenter commented 2 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 70.10%. Comparing base (52ba813) to head (8b2ef2e). Report is 85 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #2875 +/- ## ============================================ + Coverage 64.24% 70.10% +5.86% - Complexity 10398 11795 +1397 ============================================ Files 840 851 +11 Lines 71755 72609 +854 Branches 8611 8737 +126 ============================================ + Hits 46099 50904 +4805 + Misses 23004 19082 -3922 + Partials 2652 2623 -29 ```

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