Repository: incubator-gobblin
Updated Branches:
  refs/heads/master 6618c7f72 -> 75f725eda


[GOBBLIN-455] Fix handling of jvmflags and jars flags in gobblin-yarn launcher

Fix handling of jvmflags and jars flags in
gobblin-yarn launcher

Remove shift on action

Closes #2325 from DanSedov/patch-1


Project: http://git-wip-us.apache.org/repos/asf/incubator-gobblin/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-gobblin/commit/75f725ed
Tree: http://git-wip-us.apache.org/repos/asf/incubator-gobblin/tree/75f725ed
Diff: http://git-wip-us.apache.org/repos/asf/incubator-gobblin/diff/75f725ed

Branch: refs/heads/master
Commit: 75f725eda327c67b02940fa4a2f9c577fa40a1e1
Parents: 6618c7f
Author: DanSedov <[email protected]>
Authored: Fri May 4 10:56:27 2018 -0700
Committer: Abhishek Tiwari <[email protected]>
Committed: Fri May 4 10:56:27 2018 -0700

----------------------------------------------------------------------
 bin/gobblin-yarn.sh | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-gobblin/blob/75f725ed/bin/gobblin-yarn.sh
----------------------------------------------------------------------
diff --git a/bin/gobblin-yarn.sh b/bin/gobblin-yarn.sh
index c08ef48..232e719 100755
--- a/bin/gobblin-yarn.sh
+++ b/bin/gobblin-yarn.sh
@@ -77,14 +77,13 @@ do
   case "$1" in
     start|stop)
       ACTION="$1"
-      shift
       ;;
     --jvmflags)
-      JVM_FLAGS="$1"
+      JVM_FLAGS="$2"
       shift
       ;;
     --jars)
-      EXTRA_JARS="$1"
+      EXTRA_JARS="$2"
       shift
       ;;
     --help)
@@ -102,7 +101,7 @@ if [ -z "$JAVA_HOME" ]; then
 fi
 
 # User defined JVM flags overrides $GOBBLIN_JVM_FLAGS (if any)
-if [ -n "$JVM_FLAGS" ]; then
+if [ -z "$JVM_FLAGS" ]; then
   JVM_FLAGS="-Xmx1g -Xms512m"
 fi
 

Reply via email to