Deegue commented on PR #3756: URL: https://github.com/apache/celeborn/pull/3756#issuecomment-4976555638
> @Deegue, please discuss this in comments, instead of offline discussion. Otherwise, reviewers and others could not know the input of this change. BTW, please update the title and description according to updates. Thanks @SteNicholas, PR description updated. Previously we discussed if it's necessary to add a rate limit and interval to call `getPinnedMemory` which iterates all pooled Netty allocators. Since it can add noticeable CPU/alloc overhead at this frequency after hitting the high watermark, we decided to add a cache for that. -- 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]
