address comments

Author: liuxiaocs7

hbase-compression/hbase-compression-zstd/src/test/java/org/apache/hadoop/hbase/io/compress/zstd/TestHFileCompressionZstd.java

@@ -81,7 +81,7 @@ public void testReconfLevels() throws Exception {
     long len_2 = FS.getFileStatus(path_2).getLen();
     LOG.info("Level 1 len {}", len_1);
     LOG.info("Level 22 len {}", len_2);
-    assertTrue(len_1 > len_2, "Reconfiguraton with ZSTD_LEVEL_KEY did not seem to work");
+    assertTrue(len_1 > len_2, "Reconfiguration with ZSTD_LEVEL_KEY did not seem to work");
   }
 
 }

hbase-server/src/test/java/org/apache/hadoop/hbase/io/compress/HFileTestBase.java

@@ -82,38 +82,29 @@ public void doTest(Configuration conf, Path path, Compression.Algorithm compress
     // read it back in
     LOG.info("Reading with " + fileContext);
     int i = 0;
-    HFileScanner scanner = null;
-    HFile.Reader reader = HFile.createReader(FS, path, cacheConf, true, conf);
-    try {
-      scanner = reader.getScanner(conf, false, false);
+    try (HFile.Reader reader = HFile.createReader(FS, path, cacheConf, true, conf);
+      HFileScanner scanner = reader.getScanner(conf, false, false)) {
       assertTrue(scanner.seekTo(), "Initial seekTo failed");
       do {
         ExtendedCell kv = scanner.getCell();
         assertTrue(testKvs.contains(KeyValueUtil.ensureKeyValue(kv)),
           "Read back an unexpected or invalid KV");
         i++;
       } while (scanner.next());
-    } finally {
-      reader.close();
-      scanner.close();
     }
 
     assertEquals(i, testKvs.size(), "Did not read back as many KVs as written");
 
     // Test random seeks with pread
     Random rand = ThreadLocalRandom.current();
     LOG.info("Random seeking with " + fileContext);
-    reader = HFile.createReader(FS, path, cacheConf, true, conf);
-    try {
-      scanner = reader.getScanner(conf, false, true);
+    try (HFile.Reader reader = HFile.createReader(FS, path, cacheConf, true, conf);
+      HFileScanner scanner = reader.getScanner(conf, false, true)) {
       assertTrue(scanner.seekTo(), "Initial seekTo failed");
       for (i = 0; i < 100; i++) {
         KeyValue kv = testKvs.get(rand.nextInt(testKvs.size()));
         assertEquals(0, scanner.seekTo(kv), "Unable to find KV as expected: " + kv);
       }
-    } finally {
-      scanner.close();
-      reader.close();
     }
   }