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

Change subject: IMPALA-8984: Fix race condition in creating Kudu table
......................................................................


Patch Set 9:

(6 comments)

http://gerrit.cloudera.org:8080/#/c/14319/9/tests/custom_cluster/test_concurrent_kudu_create.py
File tests/custom_cluster/test_concurrent_kudu_create.py:

http://gerrit.cloudera.org:8080/#/c/14319/9/tests/custom_cluster/test_concurrent_kudu_create.py@27
PS9, Line 27: class TestConcurrentKuduCreate(CustomClusterTestSuite):
flake8: E302 expected 2 blank lines, found 1


http://gerrit.cloudera.org:8080/#/c/14319/9/tests/custom_cluster/test_concurrent_kudu_create.py@46
PS9, Line 46: P
flake8: W291 trailing whitespace


http://gerrit.cloudera.org:8080/#/c/14319/9/tests/custom_cluster/test_concurrent_kudu_create.py@46
PS9, Line 46:         query = r'''create table if not exists %s(id INT,PRIMARY 
KEY(id))
line has trailing whitespace


http://gerrit.cloudera.org:8080/#/c/14319/9/tests/custom_cluster/test_concurrent_kudu_create.py@93
PS9, Line 93:
flake8: W293 blank line contains whitespace


http://gerrit.cloudera.org:8080/#/c/14319/9/tests/custom_cluster/test_concurrent_kudu_create.py@93
PS9, Line 93:
flake8: W391 blank line at end of file


http://gerrit.cloudera.org:8080/#/c/14319/9/tests/custom_cluster/test_concurrent_kudu_create.py@93
PS9, Line 93:
line has trailing whitespace



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1a4047bcdaa6b346765b96e8c36bb747a2b0091d
Gerrit-Change-Number: 14319
Gerrit-PatchSet: 9
Gerrit-Owner: wangsheng <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-Reviewer: wangsheng <[email protected]>
Gerrit-Comment-Date: Wed, 13 Nov 2019 10:31:08 +0000
Gerrit-HasComments: Yes

Reply via email to