[GitHub] [hadoop] tasanuma commented on pull request #2907: HDFS-15975. Use LongAdder instead of AtomicLong

2021-04-17 Thread GitBox
tasanuma commented on pull request #2907: URL: https://github.com/apache/hadoop/pull/2907#issuecomment-821784876 @tomscut I think this is a good optimization. Would you mind creating another PR for branch-3.3? -- This is an automated message from the Apache Git Service. To respond to

[GitHub] [hadoop] tasanuma commented on pull request #2907: HDFS-15975. Use LongAdder instead of AtomicLong

2021-04-17 Thread GitBox
tasanuma commented on pull request #2907: URL: https://github.com/apache/hadoop/pull/2907#issuecomment-821784149 Merged it. Thanks for your contribution, @tomscut. Thanks for your reviews, @virajjasani. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [hadoop] tasanuma commented on pull request #2907: HDFS-15975. Use LongAdder instead of AtomicLong

2021-04-15 Thread GitBox
tasanuma commented on pull request #2907: URL: https://github.com/apache/hadoop/pull/2907#issuecomment-820568403 > Nice one. For metrics and statistics, LongAdder provides much better throughput. Agreed with @virajjasani. Thanks for working on this, @tomscut. -- This is an