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

orpiske pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel.git

commit 95837db2edb694c016407fd90f9921069eecf34a
Author: Otavio Rodolfo Piske <[email protected]>
AuthorDate: Mon Aug 26 17:14:41 2024 +0200

    (chores) camel-core: fixed overly complex log messages
---
 .../main/java/org/apache/camel/impl/engine/AbstractCamelContext.java  | 2 +-
 .../apache/camel/processor/SplitterParallelAggregateManualTest.java   | 2 +-
 .../java/org/apache/camel/support/RouteWatcherReloadStrategy.java     | 2 +-
 .../java/org/apache/camel/support/component/ApiConsumerHelper.java    | 4 ++--
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git 
a/core/camel-base-engine/src/main/java/org/apache/camel/impl/engine/AbstractCamelContext.java
 
b/core/camel-base-engine/src/main/java/org/apache/camel/impl/engine/AbstractCamelContext.java
index de111c097a8..e12bd58a0b0 100644
--- 
a/core/camel-base-engine/src/main/java/org/apache/camel/impl/engine/AbstractCamelContext.java
+++ 
b/core/camel-base-engine/src/main/java/org/apache/camel/impl/engine/AbstractCamelContext.java
@@ -2696,7 +2696,7 @@ public abstract class AbstractCamelContext extends 
BaseService
             if (disabled > 0) {
                 sj.add("disabled:" + disabled);
             }
-            LOG.info(String.format("Routes startup (%s)", sj));
+            LOG.info("Routes startup ({})", sj);
             // if we are default/verbose then log each route line
             if (startupSummaryLevel == StartupSummaryLevel.Default || 
startupSummaryLevel == StartupSummaryLevel.Verbose) {
                 for (String line : lines) {
diff --git 
a/core/camel-core/src/test/java/org/apache/camel/processor/SplitterParallelAggregateManualTest.java
 
b/core/camel-core/src/test/java/org/apache/camel/processor/SplitterParallelAggregateManualTest.java
index 3a0990af5d8..2d95069a2ad 100644
--- 
a/core/camel-core/src/test/java/org/apache/camel/processor/SplitterParallelAggregateManualTest.java
+++ 
b/core/camel-core/src/test/java/org/apache/camel/processor/SplitterParallelAggregateManualTest.java
@@ -85,7 +85,7 @@ public class SplitterParallelAggregateManualTest extends 
ContextTestSupport {
             }
             stopWatch.taken();
 
-            log.info(String.format("test%d.%s=%d\n", numberOfRequests, 
endpoint, stopWatch.taken()));
+            log.info("test{}.{}={}\n", numberOfRequests, endpoint, 
stopWatch.taken());
         }
     }
 
diff --git 
a/core/camel-support/src/main/java/org/apache/camel/support/RouteWatcherReloadStrategy.java
 
b/core/camel-support/src/main/java/org/apache/camel/support/RouteWatcherReloadStrategy.java
index 0c79ca76529..e46a6c999af 100644
--- 
a/core/camel-support/src/main/java/org/apache/camel/support/RouteWatcherReloadStrategy.java
+++ 
b/core/camel-support/src/main/java/org/apache/camel/support/RouteWatcherReloadStrategy.java
@@ -319,7 +319,7 @@ public class RouteWatcherReloadStrategy extends 
FileWatcherResourceReloadStrateg
                         lines.add(String.format("    %s %s (%s) (source: %s)", 
status, id, uri, loc));
                     }
                 }
-                LOG.info(String.format("Routes reloaded summary (total:%s 
started:%s)", total, started));
+                LOG.info("Routes reloaded summary (total:{} started:{})", 
total, started);
                 // if we are default/verbose then log each route line
                 if (getCamelContext().getStartupSummaryLevel() == 
StartupSummaryLevel.Default
                         || getCamelContext().getStartupSummaryLevel() == 
StartupSummaryLevel.Verbose) {
diff --git 
a/core/camel-support/src/main/java/org/apache/camel/support/component/ApiConsumerHelper.java
 
b/core/camel-support/src/main/java/org/apache/camel/support/component/ApiConsumerHelper.java
index 5bba302aa1a..f9dc8118a02 100644
--- 
a/core/camel-support/src/main/java/org/apache/camel/support/component/ApiConsumerHelper.java
+++ 
b/core/camel-support/src/main/java/org/apache/camel/support/component/ApiConsumerHelper.java
@@ -69,8 +69,8 @@ public final class ApiConsumerHelper {
             result = filteredMethods.get(0);
         } else {
             result = ApiMethodHelper.getHighestPriorityMethod(filteredMethods);
-            LOG.warn(String.format("Using highest priority operation %s from 
operations %s for endpoint %s",
-                    result, filteredMethods, endpoint.getEndpointUri()));
+            LOG.warn("Using highest priority operation {} from operations {} 
for endpoint {}", result, filteredMethods,
+                    endpoint.getEndpointUri());
         }
 
         return result;

Reply via email to