You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by al...@apache.org on 2020/04/07 18:38:03 UTC
[flink] branch master updated: [FLINK-17010] Remove --quiet from s3
get operation
This is an automated email from the ASF dual-hosted git repository.
aljoscha pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/flink.git
The following commit(s) were added to refs/heads/master by this push:
new a9f3e2f [FLINK-17010] Remove --quiet from s3 get operation
a9f3e2f is described below
commit a9f3e2ffa17967c965a7d5b2cc91abe0e8a61ef8
Author: Aljoscha Krettek <al...@apache.org>
AuthorDate: Tue Apr 7 20:27:53 2020 +0200
[FLINK-17010] Remove --quiet from s3 get operation
The reason this failed with --quiet is that the test redirects stdout to
get the sorted results:
get_complete_result > "${TEST_DATA_DIR}/complete_result"
5f2f7d637d8073b1034f8c1124f52604745936eb removed the --quiet parameter
from the s3 fetch command, which changed what is printed to stdout. The
s3 command is printing which file it is downloading and this log message
also ends up in the "result" output that is being sorted and checked.
---
flink-end-to-end-tests/test-scripts/common_s3_operations.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/flink-end-to-end-tests/test-scripts/common_s3_operations.sh b/flink-end-to-end-tests/test-scripts/common_s3_operations.sh
index 7c8745e..f784c7d 100644
--- a/flink-end-to-end-tests/test-scripts/common_s3_operations.sh
+++ b/flink-end-to-end-tests/test-scripts/common_s3_operations.sh
@@ -100,7 +100,7 @@ function s3_get_by_full_path_and_filename_prefix() {
args="$args --recursive"
fi
local relative_dir=${1#$TEST_INFRA_DIR}
- aws_cli s3 cp "s3://$IT_CASE_S3_BUCKET/$2" "/hostdir/${relative_dir}" $args
+ aws_cli s3 cp --quiet "s3://$IT_CASE_S3_BUCKET/$2" "/hostdir/${relative_dir}" $args
}
###################################