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

Change subject: IMPALA-7506: support global INVALIDATE METADATA in local 
catalog mode
......................................................................


Patch Set 10:

(8 comments)

http://gerrit.cloudera.org:8080/#/c/14307/10/tests/custom_cluster/test_concurrent_ddls.py
File tests/custom_cluster/test_concurrent_ddls.py:

http://gerrit.cloudera.org:8080/#/c/14307/10/tests/custom_cluster/test_concurrent_ddls.py@170
PS10, Line 170: c
flake8: E306 expected 1 blank line before a nested definition, found 0


http://gerrit.cloudera.org:8080/#/c/14307/10/tests/custom_cluster/test_concurrent_ddls.py@175
PS10, Line 175: d
flake8: E306 expected 1 blank line before a nested definition, found 0


http://gerrit.cloudera.org:8080/#/c/14307/10/tests/custom_cluster/test_concurrent_ddls.py@189
PS10, Line 189: e
flake8: F841 local variable 'e' is assigned to but never used


http://gerrit.cloudera.org:8080/#/c/14307/10/tests/stress/test_ddl_stress.py
File tests/stress/test_ddl_stress.py:

http://gerrit.cloudera.org:8080/#/c/14307/10/tests/stress/test_ddl_stress.py@18
PS10, Line 18: import time
flake8: F401 'time' imported but unused


http://gerrit.cloudera.org:8080/#/c/14307/10/tests/stress/test_ddl_stress.py@20
PS10, Line 20: from random import random
flake8: F401 'random.random' imported but unused


http://gerrit.cloudera.org:8080/#/c/14307/10/tests/stress/test_ddl_stress.py@22
PS10, Line 22: from tests.beeswax.impala_beeswax import ImpalaBeeswaxException
flake8: F401 'tests.beeswax.impala_beeswax.ImpalaBeeswaxException' imported but 
unused


http://gerrit.cloudera.org:8080/#/c/14307/10/tests/stress/test_ddl_stress.py@23
PS10, Line 23: from tests.common.impala_test_suite import ImpalaTestSuite, 
IMPALAD_HOST_PORT_LIST
flake8: F401 'tests.common.impala_test_suite.IMPALAD_HOST_PORT_LIST' imported 
but unused


http://gerrit.cloudera.org:8080/#/c/14307/10/tests/stress/test_ddl_stress.py@96
PS10, Line 96:
flake8: W391 blank line at end of file



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib61a7ab1ffa062620ffbc2dadc34bd7a8ca9e549
Gerrit-Change-Number: 14307
Gerrit-PatchSet: 10
Gerrit-Owner: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Quanlong Huang <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Vihang Karajgaonkar <[email protected]>
Gerrit-Comment-Date: Tue, 05 Nov 2019 16:02:15 +0000
Gerrit-HasComments: Yes

Reply via email to