You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@oozie.apache.org by an...@apache.org on 2018/05/04 10:25:41 UTC

oozie git commit: OOZIE-2934 [sharelib/spark] Fix Findbugs error (Jan Hentschel via andras.piros)

Repository: oozie
Updated Branches:
  refs/heads/master 3269decd2 -> 6ac783009


OOZIE-2934 [sharelib/spark] Fix Findbugs error (Jan Hentschel via andras.piros)


Project: http://git-wip-us.apache.org/repos/asf/oozie/repo
Commit: http://git-wip-us.apache.org/repos/asf/oozie/commit/6ac78300
Tree: http://git-wip-us.apache.org/repos/asf/oozie/tree/6ac78300
Diff: http://git-wip-us.apache.org/repos/asf/oozie/diff/6ac78300

Branch: refs/heads/master
Commit: 6ac783009bd6ec0efb4607923f1df278f6a43080
Parents: 3269dec
Author: Andras Piros <an...@cloudera.com>
Authored: Fri May 4 12:23:00 2018 +0200
Committer: Andras Piros <an...@cloudera.com>
Committed: Fri May 4 12:25:07 2018 +0200

----------------------------------------------------------------------
 release-log.txt                                             | 1 +
 .../main/java/org/apache/oozie/action/hadoop/SparkMain.java | 9 +++++++--
 2 files changed, 8 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/oozie/blob/6ac78300/release-log.txt
----------------------------------------------------------------------
diff --git a/release-log.txt b/release-log.txt
index 0c52749..896c7e5 100644
--- a/release-log.txt
+++ b/release-log.txt
@@ -1,5 +1,6 @@
 -- Oozie 5.1.0 release (trunk - unreleased)
 
+OOZIE-2934 [sharelib/spark] Fix Findbugs error (Jan Hentschel via andras.piros)
 OOZIE-3225 [Kerberos] Webconsole disabled message shown instead of 401 error (kmarton via andras.piros)
 OOZIE-2494 Cron syntax not handling DST properly (kmarton via andras.piros)
 OOZIE-2427 [Kerberos] Authentication failure for the javascript resources under /ext-2.2 (lianggz via andras.piros)

http://git-wip-us.apache.org/repos/asf/oozie/blob/6ac78300/sharelib/spark/src/main/java/org/apache/oozie/action/hadoop/SparkMain.java
----------------------------------------------------------------------
diff --git a/sharelib/spark/src/main/java/org/apache/oozie/action/hadoop/SparkMain.java b/sharelib/spark/src/main/java/org/apache/oozie/action/hadoop/SparkMain.java
index d12df18..bef08e8 100644
--- a/sharelib/spark/src/main/java/org/apache/oozie/action/hadoop/SparkMain.java
+++ b/sharelib/spark/src/main/java/org/apache/oozie/action/hadoop/SparkMain.java
@@ -119,8 +119,13 @@ public class SparkMain extends LauncherMain {
     private void createPySparkLibFolder() throws OozieActionConfiguratorException, IOException {
         final File pythonLibDir = new File("python/lib");
         if(!pythonLibDir.exists()){
-            pythonLibDir.mkdirs();
-            System.out.println("PySpark lib folder " + pythonLibDir.getAbsolutePath() + " folder created.");
+            boolean pythonLibDirCreated = pythonLibDir.mkdirs();
+
+            if (pythonLibDirCreated) {
+                System.out.println("PySpark lib folder " + pythonLibDir.getAbsolutePath() + " folder created.");
+            } else {
+                System.out.println("PySpark lib folder not created at " + pythonLibDir.getAbsolutePath());
+            }
         }
 
         for(final Pattern fileNamePattern : PYSPARK_DEP_FILE_PATTERN) {