Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-3188

Review locking around AssignmentManager.regionPlans

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Not A Problem
    • Affects Version/s: 0.90.0
    • Fix Version/s: None
    • Component/s: master
    • Labels:
      None

      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?

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                streamy Jonathan Gray
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: