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

maobaolong pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git


The following commit(s) were added to refs/heads/master by this push:
     new 74366d580 [MINOR] fix(test): Disable System exit for HadoopTestBase 
(#2327)
74366d580 is described below

commit 74366d580f023fd846a94578140ae56df94671ec
Author: maobaolong <[email protected]>
AuthorDate: Mon Jan 6 16:49:57 2025 +0800

    [MINOR] fix(test): Disable System exit for HadoopTestBase (#2327)
    
    ### What changes were proposed in this pull request?
    
    Disable System exit for HadoopTestBase
    
    ### Why are the changes needed?
    
    Without this, we cannot find the root cause of failure test, we can only 
find out the test process crashed.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    Existing UTs.
---
 storage/src/test/java/org/apache/uniffle/storage/HadoopTestBase.java | 3 +++
 1 file changed, 3 insertions(+)

diff --git 
a/storage/src/test/java/org/apache/uniffle/storage/HadoopTestBase.java 
b/storage/src/test/java/org/apache/uniffle/storage/HadoopTestBase.java
index 57e35a0f7..62cd61af3 100644
--- a/storage/src/test/java/org/apache/uniffle/storage/HadoopTestBase.java
+++ b/storage/src/test/java/org/apache/uniffle/storage/HadoopTestBase.java
@@ -30,6 +30,8 @@ import org.junit.jupiter.api.AfterAll;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.io.TempDir;
 
+import org.apache.uniffle.common.util.ExitUtils;
+
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
 public class HadoopTestBase implements Serializable {
@@ -42,6 +44,7 @@ public class HadoopTestBase implements Serializable {
 
   @BeforeAll
   public static void setUpHdfs(@TempDir File tempDir) throws Exception {
+    ExitUtils.disableSystemExit();
     conf = new Configuration();
     baseDir = tempDir;
     conf.set(MiniDFSCluster.HDFS_MINIDFS_BASEDIR, baseDir.getAbsolutePath());

Reply via email to