sumitagrawl commented on PR #5705: URL: https://github.com/apache/ozone/pull/5705#issuecomment-1833900327
> This is a direct and efficient solution. But wouldn't it be better if we limited the number of buckets in memory instead of just limiting the number of buckets that can be created, since it's possible that more than just buckets have this problem, and `volume` is `FULL_CACHE` as well. Thanks @xichen01 for review, bucket is full cache table, and for performance, it was cached. - most of read/write operation involves bucket access / update. making it partial cache may impact performance and have impact to most of code where assumption is done for bucket to be full cache. So IMO, its not viable to make bucket as partial cache. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
