commit 15cbe3260090738d26283a28512e61075b2ac6f0
author iiusky <sky@03sec.com> Fri Mar 25 18:36:59 2022 +0800
committer GitHub <noreply@github.com> Fri Mar 25 18:36:59 2022 +0800
tree 30fe4d23d20da0d80e9b30c3e0ee6b514fb1bcd2
parent 49b2e59625f54e29d787e94c8483980a90952014

[ZEPPELIN-5624] Check if the path directory is compliant. (#4282)

* [ZEPPELIN-5624] Check if the path directory is compliant.

zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSettingManager.java

diff --git a/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSettingManager.java b/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSettingManager.java
index 6d30006..cfdae70 100644
--- a/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSettingManager.java
+++ b/zeppelin-zengine/src/main/java/org/apache/zeppelin/interpreter/InterpreterSettingManager.java
@@ -1035,12 +1035,11 @@
         // Cluster event accepting nodes do not need to save files repeatedly
         saveToFile();
       }
+      File localRepoPath = new File(conf.getInterpreterLocalRepoPath());
+      FileUtils.deleteDirectory(new File(localRepoPath, id));
       removed = true;
     }
 
-    File localRepoDir = new File(conf.getInterpreterLocalRepoPath() + "/" + id);
-    FileUtils.deleteDirectory(localRepoDir);
-
     return removed;
   }