This is an automated email from the ASF dual-hosted git repository.

stigahuang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git

commit c729b13a18216d9bd2e870c31f9f614a32338cbd
Author: Daniel Vanko <[email protected]>
AuthorDate: Wed Feb 5 10:04:41 2025 +0100

    IMPALA-13345: Log the package/class name when changing FE log levels
    
    Before this patch a Frontend log level change on the WebUI only logged
    the new level set without the package/class name:
    
    ...logging-support.cc:181] New Java log level set: DEBUG
    
    From now on the package/class name is also logged in the same format
    as shown on the WebUI:
    
    ...logging-support.cc:181] New Java log level set: 
org.apache.impala.analysis.Analyzer : DEBUG
    
    Testing
     * tested the log changes manually on WebUI
    
    Change-Id: Icc915f2e3754f6ccad4e7633bf32e5752225336e
    Reviewed-on: http://gerrit.cloudera.org:8080/22451
    Tested-by: Impala Public Jenkins <[email protected]>
    Reviewed-by: Zoltan Borok-Nagy <[email protected]>
---
 be/src/util/logging-support.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/be/src/util/logging-support.cc b/be/src/util/logging-support.cc
index 9e2e103ad..c10e1ded2 100644
--- a/be/src/util/logging-support.cc
+++ b/be/src/util/logging-support.cc
@@ -178,7 +178,7 @@ void SetJavaLogLevelCallback(const Webserver::WebRequest& 
req, Document* documen
         "is empty.", "error", document);
     return;
   }
-  VLOG(1) << "New Java log level set: " << result;
+  VLOG(1) << "New Java log level set: " << classname->second << " : " << 
result;
   AddDocumentMember(result, "set_java_loglevel_result", document);
 }
 

Reply via email to