[HBASE-3188] Review locking around AssignmentManager.regionPlans

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Not A Problem
• Affects Version/s: 0.90.0
• Component/s: master
• Assignee: Unassigned
• Reporter: Jonathan Gray
• Created: 02/Nov/10 01:41
• Updated: 12/Jun/22 00:33
• Resolved: 08/Jun/14 22:05

Description

AssignmentManager.regionPlans is currently a ConcurrentSkipListMap so does not require synchronization. But sometimes we do multiple operations and we synchronize on it. But other times we don't synchronize on it at all.

Let's review and make sure we're doing the right thing.

Also see if we still need this AssignmentManager.updateTimers(). Don't we disable load balancer / expiration during startup or no?

Issue Links

• relates to: HBASE-7327 Assignment Timeouts: Remove the code from the master