HDFS-17172. Fix checkstytle

Author: haiyang1987

hadoop-hdfs-project/hadoop-hdfs/src/main/java/org/apache/hadoop/hdfs/server/namenode/NameNode.java

@@ -2754,35 +2754,35 @@ private String reconfigureFSNamesystemLockMetricsParameters(final String propert
     String result;
     try {
       switch (property) {
-        case DFS_NAMENODE_LOCK_DETAILED_METRICS_KEY: {
-          if (newVal != null && !newVal.equalsIgnoreCase("true") &&
-              !newVal.equalsIgnoreCase("false")) {
-            throw new IllegalArgumentException(newVal + " is not boolean value");
-          }
-          boolean enable = (newVal == null ?
-              DFS_NAMENODE_LOCK_DETAILED_METRICS_DEFAULT : Boolean.parseBoolean(newVal));
-          result = Boolean.toString(enable);
-          namesystem.setMetricsEnabled(enable);
-          break;
-        }
-        case DFS_NAMENODE_READ_LOCK_REPORTING_THRESHOLD_MS_KEY: {
-          long readLockReportingThresholdMs = (newVal == null ?
-              DFS_NAMENODE_READ_LOCK_REPORTING_THRESHOLD_MS_DEFAULT : Long.parseLong(newVal));
-          result = Long.toString(readLockReportingThresholdMs);
-          namesystem.setReadLockReportingThresholdMs(readLockReportingThresholdMs);
-          break;
-        }
-        case DFS_NAMENODE_WRITE_LOCK_REPORTING_THRESHOLD_MS_KEY: {
-          long writeLockReportingThresholdMs = (newVal == null ?
-              DFS_NAMENODE_WRITE_LOCK_REPORTING_THRESHOLD_MS_DEFAULT : Long.parseLong(newVal));
-          result = Long.toString(writeLockReportingThresholdMs);
-          namesystem.setWriteLockReportingThresholdMs(writeLockReportingThresholdMs);
-          break;
-        }
-        default: {
-          throw new IllegalArgumentException("Unexpected property " + property + " in " +
-              "reconfigureFSNamesystemLockMetricsParameters");
+      case DFS_NAMENODE_LOCK_DETAILED_METRICS_KEY: {
+        if (newVal != null && !newVal.equalsIgnoreCase("true") &&
+            !newVal.equalsIgnoreCase("false")) {
+          throw new IllegalArgumentException(newVal + " is not boolean value");
         }
+        boolean enable = (newVal == null ?
+            DFS_NAMENODE_LOCK_DETAILED_METRICS_DEFAULT : Boolean.parseBoolean(newVal));
+        result = Boolean.toString(enable);
+        namesystem.setMetricsEnabled(enable);
+        break;
+      }
+      case DFS_NAMENODE_READ_LOCK_REPORTING_THRESHOLD_MS_KEY: {
+        long readLockReportingThresholdMs = (newVal == null ?
+            DFS_NAMENODE_READ_LOCK_REPORTING_THRESHOLD_MS_DEFAULT : Long.parseLong(newVal));
+        result = Long.toString(readLockReportingThresholdMs);
+        namesystem.setReadLockReportingThresholdMs(readLockReportingThresholdMs);
+        break;
+      }
+      case DFS_NAMENODE_WRITE_LOCK_REPORTING_THRESHOLD_MS_KEY: {
+        long writeLockReportingThresholdMs = (newVal == null ?
+            DFS_NAMENODE_WRITE_LOCK_REPORTING_THRESHOLD_MS_DEFAULT : Long.parseLong(newVal));
+        result = Long.toString(writeLockReportingThresholdMs);
+        namesystem.setWriteLockReportingThresholdMs(writeLockReportingThresholdMs);
+        break;
+      }
+      default: {
+        throw new IllegalArgumentException("Unexpected property " + property + " in " +
+            "reconfigureFSNamesystemLockMetricsParameters");
+      }
       }
       LOG.info("RECONFIGURE* changed FSNamesystemLockMetricsParameters {} to {}", property, result);
       return result;

hadoop-hdfs-project/hadoop-hdfs/src/test/java/org/apache/hadoop/hdfs/tools/TestDFSAdmin.java

@@ -100,7 +100,6 @@
 import org.slf4j.LoggerFactory;
 
 import static org.apache.hadoop.hdfs.DFSConfigKeys.DFS_NAMENODE_LOCK_DETAILED_METRICS_KEY;
-import static org.apache.hadoop.hdfs.DFSConfigKeys.DFS_NAMENODE_MAX_SLOWPEER_COLLECT_NODES_KEY;
 import static org.apache.hadoop.hdfs.client.HdfsAdmin.TRASH_PERMISSION;
 import static org.hamcrest.CoreMatchers.allOf;
 import static org.hamcrest.CoreMatchers.anyOf;