use map.entry

caroliney14 · caroliney14 · commit 111c16fe4537 · 2021-01-19T22:33:35.000-08:00
diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
@@ -3557,9 +3557,9 @@ public void updateRegionsInTransitionMetrics() {
     }
     if (LOG.isDebugEnabled() && ritsOverThreshold != null && !ritsOverThreshold.isEmpty()) {
       StringBuilder sb = new StringBuilder();
-      for (Map.Entry<String, RegionState> regionName: ritsOverThreshold.entrySet()) {
-        sb.append(regionName).append(":")
-          .append(ritsOverThreshold.get(regionName).getState().name()).append("\n");
+      for (Map.Entry<String, RegionState> rit: ritsOverThreshold.entrySet()) {
+        sb.append(rit.getKey()).append(":")
+          .append(rit.getValue().getState().name()).append("\n");
       }
       sb.delete(sb.length()-1, sb.length());
       LOG.debug("RITs over threshold: " + sb.toString());

Original file line number	Diff line number	Diff line change
`@@ -3557,9 +3557,9 @@ public void updateRegionsInTransitionMetrics() {`
`3557`	`3557`	`}`
`3558`	`3558`	`if (LOG.isDebugEnabled() && ritsOverThreshold != null && !ritsOverThreshold.isEmpty()) {`
`3559`	`3559`	`StringBuilder sb = new StringBuilder();`
`3560`		`- for (Map.Entry<String, RegionState> regionName: ritsOverThreshold.entrySet()) {`
`3561`		`- sb.append(regionName).append(":")`
`3562`		`- .append(ritsOverThreshold.get(regionName).getState().name()).append("\n");`
	`3560`	`+ for (Map.Entry<String, RegionState> rit: ritsOverThreshold.entrySet()) {`
	`3561`	`+ sb.append(rit.getKey()).append(":")`
	`3562`	`+ .append(rit.getValue().getState().name()).append("\n");`
`3563`	`3563`	`}`
`3564`	`3564`	`sb.delete(sb.length()-1, sb.length());`
`3565`	`3565`	`LOG.debug("RITs over threshold: " + sb.toString());`