Wenzhe Zhou has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/20710 )

Change subject: IMPALA-12471: Add unit tests of external jdbc tables for MySQL
......................................................................


Patch Set 10:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/20710/10/tests/custom_cluster/test_ext_data_sources.py
File tests/custom_cluster/test_ext_data_sources.py:

http://gerrit.cloudera.org:8080/#/c/20710/10/tests/custom_cluster/test_ext_data_sources.py@80
PS10, Line 80: Failed to setup MySQL testing environment
We don't need "sudo" to run docker on all our Jenkins environments. It may fail 
when running the test on local environment without "sudo". Maybe we should 
change the error message to prompts user to rerun the test with "sudo" if 
setup-mysql-env.sh failed?



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I446ec3d4ebaf53c8edac0b2d181514bde587dfae
Gerrit-Change-Number: 20710
Gerrit-PatchSet: 10
Gerrit-Owner: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
Gerrit-Comment-Date: Tue, 28 Nov 2023 21:25:52 +0000
Gerrit-HasComments: Yes

Reply via email to