Quanlong Huang 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 11:

(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:
> flake8: E306 expected 1 blank line before a nested definition, found 0
Done


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


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


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 pytest
> flake8: F401 'time' imported but unused
Done


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


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


http://gerrit.cloudera.org:8080/#/c/14307/10/tests/stress/test_ddl_stress.py@23
PS10, Line 23: # Number of tables to create per thread
> flake8: F401 'tests.common.impala_test_suite.IMPALAD_HOST_PORT_LIST' import
Done


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
Done



--
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: 11
Gerrit-Owner: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Reviewer: Bharath Vissapragada <bhara...@apache.org>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Quanlong Huang <huangquanl...@gmail.com>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Vihang Karajgaonkar <vih...@cloudera.com>
Gerrit-Comment-Date: Wed, 06 Nov 2019 00:23:02 +0000
Gerrit-HasComments: Yes

Reply via email to