sumitagrawl commented on PR #9147: URL: https://github.com/apache/ozone/pull/9147#issuecomment-3427917160
@priyeshkaratha Good point for consideration of memory foot print, We can use ConcurrentHashSet, whose memory footprint for every Long element added is 56 bytes, so overall will be 900k * 56 bytes in this exampl => 50.4 MB. Performance impact is while below cases: 1. metrics publish for container count for the volume 2. container iterator for the volume 3. getting volume used bytes So earlier to get any volume specific information, it need iterate 900k times, but volume is having only 10k containers as applicable. As per test locally, if iterate 10k containers to get data, performance is approx 3 times faster. But for Size as used in metrics of volume container count, its just constant time -- 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]
