You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kyuubi.apache.org by ch...@apache.org on 2023/03/02 03:28:34 UTC

[kyuubi] branch master updated: [KYUUBI #4435] Change kyuubi.operation.result.format log level to debug

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 885d0c7bb [KYUUBI #4435] Change kyuubi.operation.result.format log level to debug
885d0c7bb is described below

commit 885d0c7bbc17ea93f652a84344e0bbc6047fd6da
Author: wxmimperio <wx...@outlook.com>
AuthorDate: Thu Mar 2 11:28:22 2023 +0800

    [KYUUBI #4435] Change kyuubi.operation.result.format log level to debug
    
    ### _Why are the changes needed?_
    
    There is no need to log kyuubi.operation.result.format information every time a statement is executed.
    
    ### _How was this patch tested?_
    - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible
    
    - [ ] Add screenshots for manual tests if appropriate
    
    - [ ] [Run test](https://kyuubi.readthedocs.io/en/master/develop_tools/testing.html#running-tests) locally before make a pull request
    
    Closes #4435 from imperio-wxm/master.
    
    Closes #4435
    
    2dd4fb744 [Cheng Pan] Update kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/KyuubiStatement.java
    e32c6fea6 [Cheng Pan] Update kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/KyuubiStatement.java
    1d9c5ed2b [wxmimperio] Change kyuubi.operation.result.format log level to debug
    
    Lead-authored-by: wxmimperio <wx...@outlook.com>
    Co-authored-by: Cheng Pan <pa...@gmail.com>
    Signed-off-by: Cheng Pan <ch...@apache.org>
---
 .../src/main/java/org/apache/kyuubi/jdbc/hive/KyuubiStatement.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/KyuubiStatement.java b/kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/KyuubiStatement.java
index b452ca6aa..cbe32eca6 100644
--- a/kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/KyuubiStatement.java
+++ b/kyuubi-hive-jdbc/src/main/java/org/apache/kyuubi/jdbc/hive/KyuubiStatement.java
@@ -212,7 +212,7 @@ public class KyuubiStatement implements SQLStatement, KyuubiLoggable {
 
     String resultFormat =
         properties.getOrDefault("__kyuubi_operation_result_format__", DEFAULT_RESULT_FORMAT);
-    LOG.info("kyuubi.operation.result.format: " + resultFormat);
+    LOG.debug("kyuubi.operation.result.format: {}", resultFormat);
     switch (resultFormat) {
       case "arrow":
         boolean timestampAsString =
@@ -275,7 +275,7 @@ public class KyuubiStatement implements SQLStatement, KyuubiLoggable {
 
     String resultFormat =
         properties.getOrDefault("__kyuubi_operation_result_format__", DEFAULT_RESULT_FORMAT);
-    LOG.info("kyuubi.operation.result.format: " + resultFormat);
+    LOG.debug("kyuubi.operation.result.format: {}", resultFormat);
     switch (resultFormat) {
       case "arrow":
         boolean timestampAsString =