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

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


The following commit(s) were added to refs/heads/master by this push:
     new e5e635fb27b [HUDI-7672] Fix the Hive server scratch dir for tests in 
hudi-utilities (#11097)
e5e635fb27b is described below

commit e5e635fb27b1b5ee20299b82a441e617e7529ffc
Author: Danny Chan <[email protected]>
AuthorDate: Fri Apr 26 14:26:26 2024 +0800

    [HUDI-7672] Fix the Hive server scratch dir for tests in hudi-utilities 
(#11097)
    
    Currently a null/hive/${user} dir would be left over when the tests 
finished,
    which introduces some permission access issues for Azure CI test reports.
---
 .../java/org/apache/hudi/utilities/testutils/UtilitiesTestBase.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/hudi-utilities/src/test/java/org/apache/hudi/utilities/testutils/UtilitiesTestBase.java
 
b/hudi-utilities/src/test/java/org/apache/hudi/utilities/testutils/UtilitiesTestBase.java
index 6a16c49a96c..175191436ad 100644
--- 
a/hudi-utilities/src/test/java/org/apache/hudi/utilities/testutils/UtilitiesTestBase.java
+++ 
b/hudi-utilities/src/test/java/org/apache/hudi/utilities/testutils/UtilitiesTestBase.java
@@ -138,7 +138,6 @@ public class UtilitiesTestBase {
 
   public static void initTestServices(boolean needsHdfs, boolean needsHive, 
boolean needsZookeeper) throws Exception {
     hadoopConf = HoodieTestUtils.getDefaultHadoopConf();
-    hadoopConf.set("hive.exec.scratchdir", System.getenv("java.io.tmpdir") + 
"/hive");
 
     if (needsHdfs) {
       hdfsTestService = new HdfsTestService(hadoopConf);
@@ -152,6 +151,7 @@ public class UtilitiesTestBase {
     }
     storage = HoodieStorageUtils.getStorage(fs);
 
+    hadoopConf.set("hive.exec.scratchdir", basePath + "/.tmp/hive");
     if (needsHive) {
       hiveTestService = new HiveTestService(hadoopConf);
       hiveServer = hiveTestService.start();

Reply via email to