Impala Public Jenkins has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/21442 )

Change subject: IMPALA-13017: Add S3 bucket endpoint parameters for the 
minicluster
......................................................................


Patch Set 1:

(5 comments)

http://gerrit.cloudera.org:8080/#/c/21442/1/bin/jenkins/release_cloud_resources.sh
File bin/jenkins/release_cloud_resources.sh:

http://gerrit.cloudera.org:8080/#/c/21442/1/bin/jenkins/release_cloud_resources.sh@48
PS1, Line 48:   # # Supply endpoint URL and region for direct request routing 
and AWS PrivateLink support
line too long (91 > 90)


http://gerrit.cloudera.org:8080/#/c/21442/1/bin/jenkins/release_cloud_resources.sh@57
PS1, Line 57: 
line has trailing whitespace


http://gerrit.cloudera.org:8080/#/c/21442/1/testdata/bin/load-test-warehouse-snapshot.sh
File testdata/bin/load-test-warehouse-snapshot.sh:

http://gerrit.cloudera.org:8080/#/c/21442/1/testdata/bin/load-test-warehouse-snapshot.sh@58
PS1, Line 58:     # Supply endpoint URL and region for direct request routing 
and AWS PrivateLink support
line too long (91 > 90)


http://gerrit.cloudera.org:8080/#/c/21442/1/testdata/bin/load-test-warehouse-snapshot.sh@59
PS1, Line 59:     AWS_CMD=(aws s3 rm --only-show-errors --recursive 
s3://${S3_BUCKET}${TEST_WAREHOUSE_DIR})
line too long (93 > 90)


http://gerrit.cloudera.org:8080/#/c/21442/1/testdata/bin/load-test-warehouse-snapshot.sh@66
PS1, Line 66:
line has trailing whitespace



--
To view, visit http://gerrit.cloudera.org:8080/21442
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8411e934a6b40fcd183bf597efbeb701a35e0db6
Gerrit-Change-Number: 21442
Gerrit-PatchSet: 1
Gerrit-Owner: Laszlo Gaal <laszlo.g...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Comment-Date: Fri, 17 May 2024 18:48:31 +0000
Gerrit-HasComments: Yes

Reply via email to