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

Change subject: IMPALA-9266: Use custom cluster test caste to replace 
CreateKuduTableWithoutHMSTest.java
......................................................................


Patch Set 2:

(6 comments)

http://gerrit.cloudera.org:8080/#/c/14962/2/tests/custom_cluster/test_kudu_table_create_without_hms.py
File tests/custom_cluster/test_kudu_table_create_without_hms.py:

http://gerrit.cloudera.org:8080/#/c/14962/2/tests/custom_cluster/test_kudu_table_create_without_hms.py@20
PS2, Line 20: import time
flake8: F401 'time' imported but unused


http://gerrit.cloudera.org:8080/#/c/14962/2/tests/custom_cluster/test_kudu_table_create_without_hms.py@23
PS2, Line 23: from multiprocessing.pool import ThreadPool
flake8: F401 'multiprocessing.pool.ThreadPool' imported but unused


http://gerrit.cloudera.org:8080/#/c/14962/2/tests/custom_cluster/test_kudu_table_create_without_hms.py@37
PS2, Line 37: ;
flake8: E703 statement ends with a semicolon


http://gerrit.cloudera.org:8080/#/c/14962/2/tests/custom_cluster/test_kudu_table_create_without_hms.py@42
PS2, Line 42: #
flake8: E265 block comment should start with '# '


http://gerrit.cloudera.org:8080/#/c/14962/2/tests/custom_cluster/test_kudu_table_create_without_hms.py@59
PS2, Line 59: ;
flake8: E703 statement ends with a semicolon


http://gerrit.cloudera.org:8080/#/c/14962/2/tests/custom_cluster/test_kudu_table_create_without_hms.py@61
PS2, Line 61:
flake8: W391 blank line at end of file



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic532574af42ed864612cf28eecee9e0416ef272c
Gerrit-Change-Number: 14962
Gerrit-PatchSet: 2
Gerrit-Owner: wangsheng <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Comment-Date: Mon, 30 Dec 2019 12:22:11 +0000
Gerrit-HasComments: Yes

Reply via email to