mehakmeet commented on PR #4352: URL: https://github.com/apache/hadoop/pull/4352#issuecomment-1162654911
Had to force push to resolve conflicts. Some changes in the latest commit: - Moved thread context initialization and config for enabling in hadoop-common. - Weak ref thread map for iostats context, have a doubt here regarding this whether having context and the iostats as weak ref would cause any issues, since both are now mapped to thread IDs. Thinking in terms of one context per s3afs with iostats per thread. - Added new test - Disabling iostats returns EmptyIOstatisticsStore instance with no-op aggregation. -- 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]
