(hbase) branch master updated: HBASE-28448 CompressionTest hangs when run over a Ozone ofs path (#5771)

[weichiu]

This is an automated email from the ASF dual-hosted git repository.

weichiu pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/master by this push:
     new adc79a0a9c2 HBASE-28448 CompressionTest hangs when run over a Ozone 
ofs path (#5771)
adc79a0a9c2 is described below

commit adc79a0a9c2b579915a902f611a66edfddf3149c
Author: Wei-Chiu Chuang <weic...@apache.org>
AuthorDate: Tue Apr 9 10:55:39 2024 -0700

    HBASE-28448 CompressionTest hangs when run over a Ozone ofs path (#5771)
    
    
    This bug was found via HDDS-10564.
---
 .../apache/hadoop/hbase/util/CompressionTest.java   | 21 +++++++++++----------
 1 file changed, 11 insertions(+), 10 deletions(-)

diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/util/CompressionTest.java 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/util/CompressionTest.java
index 0870dbe6f9b..9065ebf116b 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/util/CompressionTest.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/util/CompressionTest.java
@@ -152,17 +152,18 @@ public class CompressionTest {
 
     Configuration conf = new Configuration();
     Path path = new Path(args[0]);
-    FileSystem fs = path.getFileSystem(conf);
-    if (fs.exists(path)) {
-      System.err.println("The specified path exists, aborting!");
-      System.exit(1);
-    }
+    try (FileSystem fs = path.getFileSystem(conf)) {
+      if (fs.exists(path)) {
+        System.err.println("The specified path exists, aborting!");
+        System.exit(1);
+      }
 
-    try {
-      doSmokeTest(fs, path, args[1]);
-    } finally {
-      fs.delete(path, false);
+      try {
+        doSmokeTest(fs, path, args[1]);
+      } finally {
+        fs.delete(path, false);
+      }
+      System.out.println("SUCCESS");
     }
-    System.out.println("SUCCESS");
   }
 }