ChenSammi commented on PR #3673: URL: https://github.com/apache/ozone/pull/3673#issuecomment-1281737656
@duongkame , yes, we need to share an LRU cache instance between RocksDB, for both merged RocksDB and rocksDB per container. So the cache can be fully utilized and also RocksDB memory consumption can be capped. The AtomicReference field of previous implementation is for this purpose. I would suggest bringing this part of implementation back. -- 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]
