steveloughran commented on issue #1820: HADOOP-16830. Add public IOStatistics API + S3A implementation URL: https://github.com/apache/hadoop/pull/1820#issuecomment-613967462 I've been convinced that we should just move to long off volatile for the stream statistics * it's only statistics and a bit of lag is acceptable * it's better to have non-blocking reads than blocking to log stats in different threads * and trying to be clever about volatile is doomed. Conclusion: we go with long and document in the IOStatistics API That it is better to be fast than synchronized if given a choice.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
