captainzmc edited a comment on pull request #1746: URL: https://github.com/apache/ozone/pull/1746#issuecomment-757949546
> I see OmVolumeArgs resets the quotaInBytes/quotaInNamespace to OzoneConsts.QUOTA_RESET in its builder function. We can also make this change in OmBucketArgs# builder constructor. In addition, we should also reset quota when 0 was passed in constructor method. +1 Good advice. I found there are some bugs with creating volumes and buckets when quota is specified to 0 or not. This proposal can fix these problems. I will fix it in a new commit. The previous Commit had add check in OzoneQuota#parseQuota and RpcClient#verifyCountsQuota/verifySpaceQuota. ---------------------------------------------------------------- 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]
