From 7e8671caa99071d6b187c2665c40a2b84d404271 Mon Sep 17 00:00:00 2001 From: Gian Merlino Date: Wed, 23 Oct 2024 07:27:22 -0700 Subject: [PATCH] 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. --- .../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; }