Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/19378 )
Change subject: IMPALA:11808: Added support for reload event in catalogD ...................................................................... Patch Set 3: (6 comments) http://gerrit.cloudera.org:8080/#/c/19378/3/tests/custom_cluster/test_events_custom_configs.py File tests/custom_cluster/test_events_custom_configs.py: http://gerrit.cloudera.org:8080/#/c/19378/3/tests/custom_cluster/test_events_custom_configs.py@235 PS3, Line 235: e flake8: F841 local variable 'event_id_before' is assigned to but never used http://gerrit.cloudera.org:8080/#/c/19378/3/tests/custom_cluster/test_events_custom_configs.py@238 PS3, Line 238: , flake8: E501 line too long (111 > 90 characters) http://gerrit.cloudera.org:8080/#/c/19378/3/tests/custom_cluster/test_events_custom_configs.py@240 PS3, Line 240: a flake8: E501 line too long (116 > 90 characters) http://gerrit.cloudera.org:8080/#/c/19378/3/tests/custom_cluster/test_events_custom_configs.py@242 PS3, Line 242: a flake8: E501 line too long (116 > 90 characters) http://gerrit.cloudera.org:8080/#/c/19378/3/tests/custom_cluster/test_events_custom_configs.py@243 PS3, Line 243: d flake8: E306 expected 1 blank line before a nested definition, found 0 http://gerrit.cloudera.org:8080/#/c/19378/3/tests/custom_cluster/test_events_custom_configs.py@251 PS3, Line 251: r flake8: E501 line too long (104 > 90 characters) -- To view, visit http://gerrit.cloudera.org:8080/19378 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic62d58837d356dc2113f3c0904228ac9de484136 Gerrit-Change-Number: 19378 Gerrit-PatchSet: 3 Gerrit-Owner: Sai Hemanth Gantasala <[email protected]> Gerrit-Reviewer: Abhishek Rawat <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Sai Hemanth Gantasala <[email protected]> Gerrit-Comment-Date: Thu, 22 Dec 2022 07:38:05 +0000 Gerrit-HasComments: Yes
