ambry
ambry copied to clipboard
{DRAFT} Remove quota checks from AmbrySecurityService.
We do not need quota checks in AmbrySecurityService after the implementation of bandwidth throttling is enabled. This PR removes those checks.
Codecov Report
Merging #2177 (436cf29) into master (67da981) will decrease coverage by
0.01%. The diff coverage isn/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.