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