captainzmc commented on pull request #1489:
URL: https://github.com/apache/ozone/pull/1489#issuecomment-733443607
> @captainzmc and @runzhiwang , I noticed that we also removed below
function:
>
> ```java
> protected void checkVolumeQuotaInBytes(OmVolumeArgs omVolumeArgs,
> long allocateSize) throws IOException {
> ...
> }
> ```
>
> So now where we will check the quota usage of the volume? There should be
one place to do the sum of volume buckets usage and then do the quota check.
Thanks for @linyiqun's attention. For now we no longer need to check the
Quota of Volume. Because we have ensured that all bucket quota and do not
exceed volume quota when we set bucket and volume quota. Therefore, to write a
key under a bucket, we simply check bucket quota. Volume's quota will naturally
not exceed as long as the bucket's quota checks pass.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]