JingGe commented on code in PR #24491:
URL: https://github.com/apache/flink/pull/24491#discussion_r1525296378


##########
flink-end-to-end-tests/test-scripts/common_s3_operations.sh:
##########
@@ -58,7 +64,11 @@ function aws_cli_stop() {
 if [[ $AWSCLI_CONTAINER_ID ]]; then
   aws_cli_stop
 fi
-aws_cli_start
+aws_cli_start || aws_cli_start

Review Comment:
   Will simple retry like this let the script be executed multiple times 
without proper cleanup of previous container, given the aws_cli_start is not 
Idempotent, afaiu.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to