Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/21031 )
Change subject: [WIP]Hierarchical metastore event processing ...................................................................... Patch Set 10: (6 comments) http://gerrit.cloudera.org:8080/#/c/21031/10/tests/custom_cluster/test_event_processing_perf.py File tests/custom_cluster/test_event_processing_perf.py: http://gerrit.cloudera.org:8080/#/c/21031/10/tests/custom_cluster/test_event_processing_perf.py@20 PS10, Line 20: from os import getenv flake8: F401 'os.getenv' imported but unused http://gerrit.cloudera.org:8080/#/c/21031/10/tests/custom_cluster/test_event_processing_perf.py@25 PS10, Line 25: from tests.common.skip import SkipIf, SkipIfFS flake8: F401 'tests.common.skip.SkipIf' imported but unused http://gerrit.cloudera.org:8080/#/c/21031/10/tests/custom_cluster/test_event_processing_perf.py@31 PS10, Line 31: @SkipIfFS.hive flake8: E302 expected 2 blank lines, found 1 http://gerrit.cloudera.org:8080/#/c/21031/10/tests/custom_cluster/test_event_processing_perf.py@217 PS10, Line 217: ( flake8: E225 missing whitespace around operator http://gerrit.cloudera.org:8080/#/c/21031/10/tests/util/event_processor_utils.py File tests/util/event_processor_utils.py: http://gerrit.cloudera.org:8080/#/c/21031/10/tests/util/event_processor_utils.py@41 PS10, Line 41: a flake8: E501 line too long (105 > 90 characters) http://gerrit.cloudera.org:8080/#/c/21031/10/tests/util/event_processor_utils.py@60 PS10, Line 60: = flake8: E501 line too long (95 > 90 characters) -- To view, visit http://gerrit.cloudera.org:8080/21031 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I76d8a739f9db6d40f01028bfd786a85d83f9e5d6 Gerrit-Change-Number: 21031 Gerrit-PatchSet: 10 Gerrit-Owner: Anonymous Coward <k.venureddy2...@gmail.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Comment-Date: Wed, 24 Apr 2024 15:12:22 +0000 Gerrit-HasComments: Yes