[HDFS-16082] Avoid non-atomic operations on exceptionsSinceLastBalance and failedTimesSinceLastSuccessfulBalance in Balancer

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 3.4.0, 3.3.2
• Fix Version/s: 3.4.0, 3.3.2
• Component/s: balancer
• Labels: pull-request-available
• Target Version/s: 3.4.0, 3.3.2
• Hadoop Flags: Reviewed
• Assignee: Viraj Jasani
• Reporter: Viraj Jasani
• Created: 21/Jun/21 08:58
• Updated: 27/Jan/24 02:43
• Resolved: 23/Jun/21 03:24

Description

Balancer has introduced 2 volatile int as part of HDFS-13783 namely: exceptionsSinceLastBalance and failedTimesSinceLastSuccessfulBalance. However, we are performing non-atomic operations on it. Since non-atomic operations done here mostly depend on their previous values, we should use AtomicInteger for both.

Issue Links

• links to: GitHub Pull Request #3127