[FLINK-1832] [scripts] fix start scripts for Cygwin and paths with spaces.
Project: http://git-wip-us.apache.org/repos/asf/flink/repo Commit: http://git-wip-us.apache.org/repos/asf/flink/commit/f2ee7838 Tree: http://git-wip-us.apache.org/repos/asf/flink/tree/f2ee7838 Diff: http://git-wip-us.apache.org/repos/asf/flink/diff/f2ee7838 Branch: refs/heads/release-0.8 Commit: f2ee7838be18d5d41849310edc1160eeb10118be Parents: 1cc6ee4 Author: Fabian Hueske <fhue...@apache.org> Authored: Tue Apr 7 20:59:44 2015 +0200 Committer: Fabian Hueske <fhue...@apache.org> Committed: Wed Apr 8 10:15:59 2015 +0200 ---------------------------------------------------------------------- flink-dist/src/main/flink-bin/bin/config.sh | 15 ++++++++++++--- flink-dist/src/main/flink-bin/bin/flink | 20 ++------------------ flink-dist/src/main/flink-bin/bin/jobmanager.sh | 18 +++--------------- .../src/main/flink-bin/bin/taskmanager.sh | 19 ++----------------- flink-dist/src/main/flink-bin/bin/webclient.sh | 18 ++---------------- 5 files changed, 21 insertions(+), 69 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/flink/blob/f2ee7838/flink-dist/src/main/flink-bin/bin/config.sh ---------------------------------------------------------------------- diff --git a/flink-dist/src/main/flink-bin/bin/config.sh b/flink-dist/src/main/flink-bin/bin/config.sh index 35861b8..26edda4 100755 --- a/flink-dist/src/main/flink-bin/bin/config.sh +++ b/flink-dist/src/main/flink-bin/bin/config.sh @@ -17,6 +17,18 @@ # limitations under the License. ################################################################################ +constructFlinkClassPath() { + + for jarfile in "$FLINK_LIB_DIR"/*.jar ; do + if [[ $FLINK_CLASSPATH = "" ]]; then + FLINK_CLASSPATH=$jarfile; + else + FLINK_CLASSPATH=$FLINK_CLASSPATH:$jarfile + fi + done + + echo $FLINK_CLASSPATH +} # These are used to mangle paths that are passed to java when using # cygwin. Cygwin paths are like linux paths, i.e. /path/to/somewhere @@ -187,9 +199,6 @@ fi # KEY_JOBM_HEAP_MB and KEY_TASKM_HEAP_MB for that! JVM_ARGS="" -# Default classpath -CLASSPATH=`manglePathList $( echo $FLINK_LIB_DIR/*.jar . | sed 's/ /:/g' )` - # Auxilliary function which extracts the name of host from a line which # also potentialy includes topology information and the instance type extractHostName() { http://git-wip-us.apache.org/repos/asf/flink/blob/f2ee7838/flink-dist/src/main/flink-bin/bin/flink ---------------------------------------------------------------------- diff --git a/flink-dist/src/main/flink-bin/bin/flink b/flink-dist/src/main/flink-bin/bin/flink old mode 100755 new mode 100644 index e5dd3c6..79c4a6f --- a/flink-dist/src/main/flink-bin/bin/flink +++ b/flink-dist/src/main/flink-bin/bin/flink @@ -27,27 +27,11 @@ if [ "$FLINK_IDENT_STRING" = "" ]; then FLINK_IDENT_STRING="$USER" fi - -# auxiliary function to construct a lightweight classpath for the -# Flink CLI client -constructCLIClientClassPath() { - - for jarfile in $FLINK_LIB_DIR/*.jar ; do - if [[ $CC_CLASSPATH = "" ]]; then - CC_CLASSPATH=$jarfile; - else - CC_CLASSPATH=$CC_CLASSPATH:$jarfile - fi - done - - echo $CC_CLASSPATH -} - -CC_CLASSPATH=`manglePathList $(constructCLIClientClassPath)` +CC_CLASSPATH=`constructFlinkClassPath` log=$FLINK_LOG_DIR/flink-$FLINK_IDENT_STRING-flink-client-$HOSTNAME.log log_setting="-Dlog.file="$log" -Dlog4j.configuration=file:"$FLINK_CONF_DIR"/log4j.properties -Dlogback.configurationFile=file:"$FLINK_CONF_DIR"/logback.xml" export FLINK_CONF_DIR -$JAVA_RUN $JVM_ARGS $log_setting -classpath $CC_CLASSPATH org.apache.flink.client.CliFrontend $* +$JAVA_RUN $JVM_ARGS "$log_setting" -classpath "`manglePathList "$CC_CLASSPATH"`" org.apache.flink.client.CliFrontend $* http://git-wip-us.apache.org/repos/asf/flink/blob/f2ee7838/flink-dist/src/main/flink-bin/bin/jobmanager.sh ---------------------------------------------------------------------- diff --git a/flink-dist/src/main/flink-bin/bin/jobmanager.sh b/flink-dist/src/main/flink-bin/bin/jobmanager.sh index 0a9f9f7..74ff22a 100755 --- a/flink-dist/src/main/flink-bin/bin/jobmanager.sh +++ b/flink-dist/src/main/flink-bin/bin/jobmanager.sh @@ -44,20 +44,7 @@ if [ "$FLINK_IDENT_STRING" = "" ]; then FLINK_IDENT_STRING="$USER" fi -# auxilliary function to construct a the classpath for the jobmanager -constructJobManagerClassPath() { - for jarfile in "$FLINK_LIB_DIR"/*.jar ; do - if [[ $FLINK_JM_CLASSPATH = "" ]]; then - FLINK_JM_CLASSPATH=$jarfile; - else - FLINK_JM_CLASSPATH=$FLINK_JM_CLASSPATH:$jarfile - fi - done - - echo $FLINK_JM_CLASSPATH -} - -FLINK_JM_CLASSPATH=`manglePathList "$(constructJobManagerClassPath)"` +FLINK_JM_CLASSPATH=`constructFlinkClassPath` log=$FLINK_LOG_DIR/flink-$FLINK_IDENT_STRING-jobmanager-$HOSTNAME.log out=$FLINK_LOG_DIR/flink-$FLINK_IDENT_STRING-jobmanager-$HOSTNAME.out @@ -80,7 +67,8 @@ case $STARTSTOP in rotateLogFile $out echo Starting job manager - $JAVA_RUN $JVM_ARGS ${FLINK_ENV_JAVA_OPTS} "${log_setting[@]}" -classpath "$FLINK_JM_CLASSPATH:$HADOOP_CLASSPATH" org.apache.flink.runtime.jobmanager.JobManager -executionMode $EXECUTIONMODE -configDir "$FLINK_CONF_DIR" > "$out" 2>&1 < /dev/null & + + $JAVA_RUN $JVM_ARGS ${FLINK_ENV_JAVA_OPTS} "${log_setting[@]}" -classpath "`manglePathList "$FLINK_JM_CLASSPATH:$HADOOP_CLASSPATH"`" org.apache.flink.runtime.jobmanager.JobManager -executionMode $EXECUTIONMODE -configDir "$FLINK_CONF_DIR" > "$out" 2>&1 < /dev/null & echo $! > $pid ;; http://git-wip-us.apache.org/repos/asf/flink/blob/f2ee7838/flink-dist/src/main/flink-bin/bin/taskmanager.sh ---------------------------------------------------------------------- diff --git a/flink-dist/src/main/flink-bin/bin/taskmanager.sh b/flink-dist/src/main/flink-bin/bin/taskmanager.sh index 0bf54e3..97e730e 100755 --- a/flink-dist/src/main/flink-bin/bin/taskmanager.sh +++ b/flink-dist/src/main/flink-bin/bin/taskmanager.sh @@ -29,22 +29,7 @@ if [ "$FLINK_IDENT_STRING" = "" ]; then FLINK_IDENT_STRING="$USER" fi -# auxilliary function to construct a lightweight classpath for the -# Flink TaskManager -constructTaskManagerClassPath() { - - for jarfile in "$FLINK_LIB_DIR"/*.jar ; do - if [[ $FLINK_TM_CLASSPATH = "" ]]; then - FLINK_TM_CLASSPATH=$jarfile; - else - FLINK_TM_CLASSPATH=$FLINK_TM_CLASSPATH:$jarfile - fi - done - - echo $FLINK_TM_CLASSPATH -} - -FLINK_TM_CLASSPATH=`manglePathList "$(constructTaskManagerClassPath)"` +FLINK_TM_CLASSPATH=`constructFlinkClassPath` log=$FLINK_LOG_DIR/flink-$FLINK_IDENT_STRING-taskmanager-$HOSTNAME.log out=$FLINK_LOG_DIR/flink-$FLINK_IDENT_STRING-taskmanager-$HOSTNAME.out @@ -77,7 +62,7 @@ case $STARTSTOP in rotateLogFile $out echo Starting task manager on host $HOSTNAME - $JAVA_RUN $JVM_ARGS ${FLINK_ENV_JAVA_OPTS} "${log_setting[@]}" -classpath "$FLINK_TM_CLASSPATH:$HADOOP_CLASSPATH" org.apache.flink.runtime.taskmanager.TaskManager -configDir "$FLINK_CONF_DIR" > "$out" 2>&1 < /dev/null & + $JAVA_RUN $JVM_ARGS ${FLINK_ENV_JAVA_OPTS} "${log_setting[@]}" -classpath "`manglePathList "$FLINK_TM_CLASSPATH:$HADOOP_CLASSPATH"`" org.apache.flink.runtime.taskmanager.TaskManager -configDir "$FLINK_CONF_DIR" > "$out" 2>&1 < /dev/null & echo $! > $pid ;; http://git-wip-us.apache.org/repos/asf/flink/blob/f2ee7838/flink-dist/src/main/flink-bin/bin/webclient.sh ---------------------------------------------------------------------- diff --git a/flink-dist/src/main/flink-bin/bin/webclient.sh b/flink-dist/src/main/flink-bin/bin/webclient.sh index ecf86ff..8e985f7 100755 --- a/flink-dist/src/main/flink-bin/bin/webclient.sh +++ b/flink-dist/src/main/flink-bin/bin/webclient.sh @@ -32,21 +32,7 @@ fi JVM_ARGS="$JVM_ARGS -Xmx512m" -# auxilliary function to construct the classpath for the webclient -constructWebclientClassPath() { - - for jarfile in "$FLINK_LIB_DIR"/*.jar ; do - if [[ $FLINK_WEBCLIENT_CLASSPATH = "" ]]; then - FLINK_WEBCLIENT_CLASSPATH=$jarfile; - else - FLINK_WEBCLIENT_CLASSPATH=$FLINK_WEBCLIENT_CLASSPATH:$jarfile - fi - done - - echo $FLINK_WEBCLIENT_CLASSPATH -} - -FLINK_WEBCLIENT_CLASSPATH=`manglePathList "$(constructWebclientClassPath)"` +FLINK_WEBCLIENT_CLASSPATH=`constructFlinkClassPath` log=$FLINK_LOG_DIR/flink-$FLINK_IDENT_STRING-webclient-$HOSTNAME.log out=$FLINK_LOG_DIR/flink-$FLINK_IDENT_STRING-webclient-$HOSTNAME.out @@ -64,7 +50,7 @@ case $STARTSTOP in fi fi echo Starting Flink webclient - $JAVA_RUN $JVM_ARGS "${log_setting[@]}" -classpath "$FLINK_WEBCLIENT_CLASSPATH:$HADOOP_CLASSPATH" org.apache.flink.client.WebFrontend -configDir "$FLINK_CONF_DIR" > "$out" 2>&1 < /dev/null & + $JAVA_RUN $JVM_ARGS "${log_setting[@]}" -classpath "`manglePathList "$FLINK_WEBCLIENT_CLASSPATH:$HADOOP_CLASSPATH"`" org.apache.flink.client.WebFrontend -configDir "$FLINK_CONF_DIR" > "$out" 2>&1 < /dev/null & echo $! > $pid ;;