diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/AssignmentManager.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/AssignmentManager.java index e531d612d3e..06aef4a83ec 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/AssignmentManager.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/assignment/AssignmentManager.java @@ -1176,15 +1176,9 @@ public class AssignmentManager { if (rsn.getState() != State.OPEN) { continue; // Opportunistic check, should quickly skip RITs, offline tables, etc. } - ServerName sn; - State state; - rsn.lock(); - try { - sn = rsn.getRegionLocation(); - state = rsn.getState(); - } finally { - rsn.unlock(); - } + // Do not need to acquire region state lock as this is only for showing metrics. + ServerName sn = rsn.getRegionLocation(); + State state = rsn.getState(); if (state != State.OPEN) { continue; // Mostly skipping RITs that are already being take care of. }