Bharath Vissapragada has posted comments on this change. Change subject: IMPALA-4822: Implement dynamic log level changes ......................................................................
Patch Set 1: (2 comments) PS4 implements dynamic logging changes to the backend (equivalent to setting --v on the fly) and also the "reset" functionality for both log4j and the backend. That resets the logging state as per the user defined values at the startup. Also, I'm looking for some suggestions on cosmetic changes to the log_level web end point. As of now it works, but looks pretty basic (like it is designed by someone who just learnt HTML :D). So that needs to be improved as well. http://gerrit.cloudera.org:8080/#/c/5792/1/be/src/util/logging-support.cc File be/src/util/logging-support.cc: Line 26: #include "rpc/jni-thrift-util.h" > names.h basically has a bunch of common "using" directives, so is more appr Thanks for explaining. http://gerrit.cloudera.org:8080/#/c/5792/1/common/thrift/Logging.thrift File common/thrift/Logging.thrift: Line 35: struct TGetLogLevelParams { > Agree that there is additional bookeeping. I'm generally of the opinion tha I agree with your point that having a visual feedback is much more easy to the end user. For now I've implemented the "reset" functionality. Still need to look into log4j if there is a way to get all the overrides. -- To view, visit http://gerrit.cloudera.org:8080/5792 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I588418e9bcb0b66d33138baf96207a5a35bfbd63 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Alex Behm <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Dimitris Tsirogiannis <[email protected]> Gerrit-Reviewer: Henry Robinson <[email protected]> Gerrit-HasComments: Yes
