This is an automated email from the ASF dual-hosted git repository. zhijiang pushed a commit to branch release-1.10 in repository https://gitbox.apache.org/repos/asf/flink.git
The following commit(s) were added to refs/heads/release-1.10 by this push: new 5ebab4f [FLINK-15360][e2e] Fix the issue of parameter with whitespace in retry_times shell function 5ebab4f is described below commit 5ebab4fa2e51791fc04e04e3ab6fbbfc9f243fce Author: Yangze Guo <karma...@gmail.com> AuthorDate: Mon Dec 23 22:23:45 2019 +0800 [FLINK-15360][e2e] Fix the issue of parameter with whitespace in retry_times shell function Yarn e2e test is broken with building docker image. It is because the shell function retry_times can not support the parameter with whitespace properly (E.g. `retry_times 5 0 docker build image`). We make the third parameter which might contain whitespace as a local variable and then pass it into the function to solve the issue. --- flink-end-to-end-tests/test-scripts/common.sh | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/flink-end-to-end-tests/test-scripts/common.sh b/flink-end-to-end-tests/test-scripts/common.sh index e519056..01dda22 100644 --- a/flink-end-to-end-tests/test-scripts/common.sh +++ b/flink-end-to-end-tests/test-scripts/common.sh @@ -736,7 +736,8 @@ function find_latest_completed_checkpoint { } function retry_times() { - retry_times_with_backoff_and_cleanup $1 $2 "${@:3}" "true" + local command=${@:3} + retry_times_with_backoff_and_cleanup $1 $2 "$command" "true" } function retry_times_with_backoff_and_cleanup() {