ambry icon indicating copy to clipboard operation
ambry copied to clipboard

{DRAFT} Remove quota checks from AmbrySecurityService.

Open ankagrawal opened this issue 3 years ago • 1 comments

We do not need quota checks in AmbrySecurityService after the implementation of bandwidth throttling is enabled. This PR removes those checks.

ankagrawal avatar Aug 13 '22 19:08 ankagrawal

Codecov Report

Merging #2177 (436cf29) into master (67da981) will decrease coverage by 0.01%. The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2177      +/-   ##
============================================
- Coverage     73.03%   73.01%   -0.02%     
+ Complexity    10214    10206       -8     
============================================
  Files           737      737              
  Lines         56086    56080       -6     
  Branches       6905     6902       -3     
============================================
- Hits          40960    40948      -12     
- Misses        12916    12920       +4     
- Partials       2210     2212       +2     
Impacted Files Coverage Δ
...om/github/ambry/frontend/AmbrySecurityService.java 92.03% <ø> (+0.65%) :arrow_up:
...va/com/github/ambry/rest/RestServiceException.java 88.46% <0.00%> (-11.54%) :arrow_down:
...c/main/java/com/github/ambry/quota/QuotaUtils.java 83.33% <0.00%> (-8.34%) :arrow_down:
...va/com/github/ambry/rest/NettyResponseChannel.java 90.18% <0.00%> (-0.76%) :arrow_down:
...ls/src/main/java/com/github/ambry/utils/Utils.java 72.79% <0.00%> (+0.25%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov-commenter avatar Aug 13 '22 19:08 codecov-commenter