commit b99bb28bcf6c3adb3a1f557d8217ef8624b422b4
author bd2019us <47727869+bd2019us@users.noreply.github.com> Tue Apr 02 09:53:50 2019 -0500
committer Robert Levas <rlevas@apache.org> Tue Apr 02 10:53:50 2019 -0400
tree 459fde3cb9edaf3abae2a2d3a14b38e69805c63f
parent 2bc4779a1e6aabe638101fc8b0e28cd1963d6b13

[AMBARI-25221] fix directory creation from mkdir to mkdirs to avoid crash

* [AMBARI-25221] fix directory creation from mkdir to mkdirs to avoid crash

* [AMBARI-25221] fix mkdir to mkdirs

ambari-server/src/main/java/org/apache/ambari/server/mpack/MpackManager.java

diff --git a/ambari-server/src/main/java/org/apache/ambari/server/mpack/MpackManager.java b/ambari-server/src/main/java/org/apache/ambari/server/mpack/MpackManager.java
index 6bfacee..a60e460 100644
--- a/ambari-server/src/main/java/org/apache/ambari/server/mpack/MpackManager.java
+++ b/ambari-server/src/main/java/org/apache/ambari/server/mpack/MpackManager.java
@@ -231,7 +231,7 @@
     LOG.debug("Download mpack.json and store in :" + targetPath);
 
     if (!stagingDir.exists()) {
-      stagingDir.mkdir();
+      stagingDir.mkdirs();
     }
 
     Files.copy(url.openStream(), targetPath, StandardCopyOption.REPLACE_EXISTING);
@@ -370,7 +370,7 @@
   private void createServicesDirectory(Path extractedMpackDirectory, Mpack mpack) throws IOException {
     File servicesDir = new File(extractedMpackDirectory.toAbsolutePath() + File.separator + MODULES_DIRECTORY);
     if (!servicesDir.exists()) {
-      servicesDir.mkdir();
+      servicesDir.mkdirs();
     }
     List<Module> modules = mpack.getModules();
 
@@ -405,7 +405,7 @@
           File mpackDirectory = new File(mpackStaging + File.separator + mpack.getName());
 
           if (!mpackDirectory.exists()) {
-            mpackDirectory.mkdir();
+            mpackDirectory.mkdirs();
           }
           return true;
         } else {
@@ -447,7 +447,7 @@
     URL url = new URL(mpackTarURI);
 
     if (!stagingDir.exists()) {
-      stagingDir.mkdir();
+      stagingDir.mkdirs();
     }
 
     Files.copy(url.openStream(), targetPath, StandardCopyOption.REPLACE_EXISTING);