[flink] branch release-1.10 updated: [FLINK-15360][e2e] Fix the issue of parameter with whitespace in retry_times shell function

[zhijiang]

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() {