HBase
  1. HBase
  2. HBASE-3188

Review locking around AssignmentManager.regionPlans

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Minor 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?

        Issue Links

          Activity

          Hide
          Andrew Purtell added a comment - - edited

          Stale

          Show
          Andrew Purtell added a comment - - edited Stale

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Development