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

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


The following commit(s) were added to refs/heads/master by this push:
     new 7e8671caa99 GroupByQueryConfig: Skip unnecessary toString. (#17396)
7e8671caa99 is described below

commit 7e8671caa99071d6b187c2665c40a2b84d404271
Author: Gian Merlino <[email protected]>
AuthorDate: Wed Oct 23 07:27:22 2024 -0700

    GroupByQueryConfig: Skip unnecessary toString. (#17396)
    
    Calling toString on newConfig is unnecessary, because it will be done
    automatically by the logger. This saves some effort under log levels
    higher than DEBUG.
---
 .../main/java/org/apache/druid/query/groupby/GroupByQueryConfig.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/processing/src/main/java/org/apache/druid/query/groupby/GroupByQueryConfig.java
 
b/processing/src/main/java/org/apache/druid/query/groupby/GroupByQueryConfig.java
index dbd6f2869a7..23ef9fae820 100644
--- 
a/processing/src/main/java/org/apache/druid/query/groupby/GroupByQueryConfig.java
+++ 
b/processing/src/main/java/org/apache/druid/query/groupby/GroupByQueryConfig.java
@@ -370,7 +370,7 @@ public class GroupByQueryConfig
         isMultiValueUnnestingEnabled()
     );
 
-    logger.debug("Override config for GroupBy query %s - %s", query.getId(), 
newConfig.toString());
+    logger.debug("Override config for GroupBy query %s - %s", query.getId(), 
newConfig);
     return newConfig;
   }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to