Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/15903 )
Change subject: IMPALA-8860: Improve /log_level usability on WebUI ...................................................................... Patch Set 5: (3 comments) http://gerrit.cloudera.org:8080/#/c/15903/5/be/src/util/logging-support.cc File be/src/util/logging-support.cc: http://gerrit.cloudera.org:8080/#/c/15903/5/be/src/util/logging-support.cc@142 PS5, Line 142: for (const string& logLevel: result.loglevels) { line has trailing whitespace http://gerrit.cloudera.org:8080/#/c/15903/5/tests/webserver/test_web_pages.py File tests/webserver/test_web_pages.py: http://gerrit.cloudera.org:8080/#/c/15903/5/tests/webserver/test_web_pages.py@211 PS5, Line 211: flake8: E501 line too long (98 > 90 characters) http://gerrit.cloudera.org:8080/#/c/15903/5/tests/webserver/test_web_pages.py@236 PS5, Line 236: flake8: E501 line too long (98 > 90 characters) -- To view, visit http://gerrit.cloudera.org:8080/15903 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I2fbf2ef21f4af297913a4e9b16a391768624da33 Gerrit-Change-Number: 15903 Gerrit-PatchSet: 5 Gerrit-Owner: Zoltan Garaguly <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Tamas Mate <[email protected]> Gerrit-Reviewer: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: Zoltan Garaguly <[email protected]> Gerrit-Comment-Date: Tue, 19 May 2020 12:12:52 +0000 Gerrit-HasComments: Yes
