Repository: flink
Updated Branches:
  refs/heads/master 245f02bd6 -> 6590a4c36


[FLINK-4219] [scripts] Quote PDSH opts in start-cluster.sh

This prevents word splitting if the user configures multiple SSH
options.

This closes #2253.


Project: http://git-wip-us.apache.org/repos/asf/flink/repo
Commit: http://git-wip-us.apache.org/repos/asf/flink/commit/6590a4c3
Tree: http://git-wip-us.apache.org/repos/asf/flink/tree/6590a4c3
Diff: http://git-wip-us.apache.org/repos/asf/flink/diff/6590a4c3

Branch: refs/heads/master
Commit: 6590a4c364dafeb9aae84ee1f0b57782ba237031
Parents: 245f02b
Author: Greg Hogan <c...@greghogan.com>
Authored: Thu Jul 14 13:40:35 2016 -0400
Committer: Ufuk Celebi <u...@apache.org>
Committed: Fri Aug 5 10:31:38 2016 +0200

----------------------------------------------------------------------
 flink-dist/src/main/flink-bin/bin/start-cluster.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/flink/blob/6590a4c3/flink-dist/src/main/flink-bin/bin/start-cluster.sh
----------------------------------------------------------------------
diff --git a/flink-dist/src/main/flink-bin/bin/start-cluster.sh 
b/flink-dist/src/main/flink-bin/bin/start-cluster.sh
index a22eb8d..77bff1e 100755
--- a/flink-dist/src/main/flink-bin/bin/start-cluster.sh
+++ b/flink-dist/src/main/flink-bin/bin/start-cluster.sh
@@ -58,6 +58,6 @@ if [[ $? -ne 0 ]]; then
         ssh -n $FLINK_SSH_OPTS $slave -- "nohup /bin/bash -l 
\"${FLINK_BIN_DIR}/taskmanager.sh\" start &"
     done
 else
-    PDSH_SSH_ARGS="" PDSH_SSH_ARGS_APPEND=$FLINK_SSH_OPTS pdsh -w $(IFS=, ; 
echo "${SLAVES[*]}") \
+    PDSH_SSH_ARGS="" PDSH_SSH_ARGS_APPEND="${FLINK_SSH_OPTS}" pdsh -w $(IFS=, 
; echo "${SLAVES[*]}") \
         "nohup /bin/bash -l \"${FLINK_BIN_DIR}/taskmanager.sh\" start"
 fi

Reply via email to