You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by or...@apache.org on 2022/06/20 12:22:22 UTC

[camel] branch main updated (857e0abe6e0 -> 319ce436572)

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

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


    from 857e0abe6e0 (chores) openapi-rest-dsl-generator: removed unused parameters
     new 4175a140a2f (chores) camel-debug: use format specifiers in log messages
     new 98c13b5672d (chores) camel-google-bigquery: remove unnecessary toString()
     new bd5e1b34235 (chores) camel-jira: use format specifiers in log messages
     new 3dc7d795d68 (chores) camel-mock: remove unnecessary toString()
     new 3d3107d1353 (chores) camel-rabbitmq: remove unnecessary toString()
     new 72ed2803475 (chores) camel-test-spring-junit5: use format specifiers in log messages
     new a8c19be2c39 (chores) camel-base-engine: use format specifiers in log messages
     new 319ce436572 (chores) camel-console: use format specifiers in log messages

The 8 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../java/org/apache/camel/component/debug/JmxConnectorService.java   | 3 +--
 .../component/google/bigquery/sql/GoogleBigQuerySQLProducer.java     | 2 +-
 .../apache/camel/component/jira/consumer/AbstractJiraConsumer.java   | 4 ++--
 .../org/apache/camel/component/jira/consumer/NewIssuesConsumer.java  | 5 +++--
 .../src/main/java/org/apache/camel/component/mock/TimeClause.java    | 2 +-
 .../apache/camel/component/rabbitmq/RabbitMQMessageConverter.java    | 2 +-
 .../org/apache/camel/test/spring/junit5/RouteCoverageDumper.java     | 2 +-
 .../main/java/org/apache/camel/impl/engine/AbstractCamelContext.java | 4 ++--
 .../java/org/apache/camel/impl/console/DefaultDevConsolesLoader.java | 4 ++--
 9 files changed, 14 insertions(+), 14 deletions(-)


[camel] 07/08: (chores) camel-base-engine: use format specifiers in log messages

Posted by or...@apache.org.
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 a8c19be2c39ce0048b080c04042a2f05322b1131
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Mon Jun 20 11:43:36 2022 +0200

    (chores) camel-base-engine: use format specifiers in log messages
---
 .../main/java/org/apache/camel/impl/engine/AbstractCamelContext.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 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 bb5b135f9dc..6a82ec38327 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
@@ -3060,8 +3060,8 @@ public abstract class AbstractCamelContext extends BaseService
 
         if (adapt(ExtendedCamelContext.class).getExchangeFactory().isPooled()) {
             LOG.info(
-                    "Pooled mode enabled. Camel pools and reuses objects to reduce JVM object allocations. The pool capacity is: "
-                     + adapt(ExtendedCamelContext.class).getExchangeFactory().getCapacity() + " elements.");
+                    "Pooled mode enabled. Camel pools and reuses objects to reduce JVM object allocations. The pool capacity is: {} elements.",
+                    adapt(ExtendedCamelContext.class).getExchangeFactory().getCapacity());
         }
         if (isLightweight()) {
             LOG.info("Lightweight mode enabled. Performing optimizations and memory reduction.");


[camel] 05/08: (chores) camel-rabbitmq: remove unnecessary toString()

Posted by or...@apache.org.
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 3d3107d135331ffe02f3d89ecc76fd6f0d4bfd04
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Mon Jun 20 11:40:06 2022 +0200

    (chores) camel-rabbitmq: remove unnecessary toString()
---
 .../org/apache/camel/component/rabbitmq/RabbitMQMessageConverter.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/components/camel-rabbitmq/src/main/java/org/apache/camel/component/rabbitmq/RabbitMQMessageConverter.java b/components/camel-rabbitmq/src/main/java/org/apache/camel/component/rabbitmq/RabbitMQMessageConverter.java
index 7fe26390325..916e95d3151 100644
--- a/components/camel-rabbitmq/src/main/java/org/apache/camel/component/rabbitmq/RabbitMQMessageConverter.java
+++ b/components/camel-rabbitmq/src/main/java/org/apache/camel/component/rabbitmq/RabbitMQMessageConverter.java
@@ -214,7 +214,7 @@ public class RabbitMQMessageConverter {
                 .filter(key -> !RabbitMQConstants.BASIC_AMQP_PROPERTIES.contains(key))
                 .collect(Collectors.joining(", "));
         if (LOG.isDebugEnabled()) {
-            LOG.debug("Added AMQP basic properties: {}", properties.build().toString());
+            LOG.debug("Added AMQP basic properties: {}", properties.build());
             LOG.debug("Ignoring non-AMQP basic properties: {}", ignoredProperties);
         }
 


[camel] 03/08: (chores) camel-jira: use format specifiers in log messages

Posted by or...@apache.org.
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 bd5e1b34235a45e35b0969863bffa0ea7a62200e
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Mon Jun 20 11:38:31 2022 +0200

    (chores) camel-jira: use format specifiers in log messages
---
 .../apache/camel/component/jira/consumer/AbstractJiraConsumer.java   | 4 ++--
 .../org/apache/camel/component/jira/consumer/NewIssuesConsumer.java  | 5 +++--
 2 files changed, 5 insertions(+), 4 deletions(-)

diff --git a/components/camel-jira/src/main/java/org/apache/camel/component/jira/consumer/AbstractJiraConsumer.java b/components/camel-jira/src/main/java/org/apache/camel/component/jira/consumer/AbstractJiraConsumer.java
index f71fc95991c..d19fbd9817f 100644
--- a/components/camel-jira/src/main/java/org/apache/camel/component/jira/consumer/AbstractJiraConsumer.java
+++ b/components/camel-jira/src/main/java/org/apache/camel/component/jira/consumer/AbstractJiraConsumer.java
@@ -61,8 +61,8 @@ public abstract class AbstractJiraConsumer extends ScheduledPollConsumer {
                     int code = rcr.getStatusCode().get();
                     // if auth or server error then cause a re-connect
                     if (code >= 400) {
-                        LOG.warn("RestClientException error code: " + code + " caused by " + rcr.getMessage()
-                                 + ". Will re-connect on next poll.");
+                        LOG.warn("RestClientException error code: {} caused by {}. Will re-connect on next poll.", code,
+                                rcr.getMessage());
                         getEndpoint().disconnect();
                     }
                 }
diff --git a/components/camel-jira/src/main/java/org/apache/camel/component/jira/consumer/NewIssuesConsumer.java b/components/camel-jira/src/main/java/org/apache/camel/component/jira/consumer/NewIssuesConsumer.java
index 6818f809b1e..4b9459502e0 100644
--- a/components/camel-jira/src/main/java/org/apache/camel/component/jira/consumer/NewIssuesConsumer.java
+++ b/components/camel-jira/src/main/java/org/apache/camel/component/jira/consumer/NewIssuesConsumer.java
@@ -100,8 +100,9 @@ public class NewIssuesConsumer extends AbstractJiraConsumer {
                 if (code == 400) {
                     String msg = e.getMessage();
                     if (msg != null && msg.contains("does not exist for the field 'id'")) {
-                        LOG.warn("Last issue id: " + latestIssueId + " no longer exists (could have been deleted)."
-                                 + " Will recover by fetching last issue id from JIRA and try again on next poll");
+                        LOG.warn("Last issue id: {} no longer exists (could have been deleted)."
+                                 + " Will recover by fetching last issue id from JIRA and try again on next poll",
+                                latestIssueId);
                         latestIssueId = findLatestIssueId();
                         return Collections.EMPTY_LIST;
                     }


[camel] 01/08: (chores) camel-debug: use format specifiers in log messages

Posted by or...@apache.org.
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 4175a140a2f651fd1ab29a8aa48715383fa871ff
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Mon Jun 20 11:35:33 2022 +0200

    (chores) camel-debug: use format specifiers in log messages
---
 .../java/org/apache/camel/component/debug/JmxConnectorService.java     | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/components/camel-debug/src/main/java/org/apache/camel/component/debug/JmxConnectorService.java b/components/camel-debug/src/main/java/org/apache/camel/component/debug/JmxConnectorService.java
index dd0c61214f8..2d2cb7a9b6e 100644
--- a/components/camel-debug/src/main/java/org/apache/camel/component/debug/JmxConnectorService.java
+++ b/components/camel-debug/src/main/java/org/apache/camel/component/debug/JmxConnectorService.java
@@ -89,8 +89,7 @@ public class JmxConnectorService extends ServiceSupport implements CamelContextA
                 cs.stop();
                 LOG.debug("Stopped JMX Connector");
             } catch (IOException e) {
-                LOG.debug("Error occurred during stopping JMXConnectorService: "
-                          + cs + ". This exception will be ignored.");
+                LOG.debug("Error occurred during stopping JMXConnectorService: {}. This exception will be ignored.", cs);
             }
             cs = null;
         }


[camel] 02/08: (chores) camel-google-bigquery: remove unnecessary toString()

Posted by or...@apache.org.
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 98c13b5672d8223f46e76fedfa96809042e38c9b
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Mon Jun 20 11:36:54 2022 +0200

    (chores) camel-google-bigquery: remove unnecessary toString()
---
 .../camel/component/google/bigquery/sql/GoogleBigQuerySQLProducer.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/components/camel-google/camel-google-bigquery/src/main/java/org/apache/camel/component/google/bigquery/sql/GoogleBigQuerySQLProducer.java b/components/camel-google/camel-google-bigquery/src/main/java/org/apache/camel/component/google/bigquery/sql/GoogleBigQuerySQLProducer.java
index 883d90c7522..da98cca893d 100644
--- a/components/camel-google/camel-google-bigquery/src/main/java/org/apache/camel/component/google/bigquery/sql/GoogleBigQuerySQLProducer.java
+++ b/components/camel-google/camel-google-bigquery/src/main/java/org/apache/camel/component/google/bigquery/sql/GoogleBigQuerySQLProducer.java
@@ -111,7 +111,7 @@ public class GoogleBigQuerySQLProducer extends DefaultProducer {
             long numAffectedRows = statistics.getNumDmlAffectedRows();
 
             if (LOG.isTraceEnabled()) {
-                LOG.trace("Query {} - Affected rows {} - Result {}", translatedQuery, numAffectedRows, result.toString());
+                LOG.trace("Query {} - Affected rows {} - Result {}", translatedQuery, numAffectedRows, result);
             }
             return numAffectedRows;
         } catch (JobException e) {


[camel] 06/08: (chores) camel-test-spring-junit5: use format specifiers in log messages

Posted by or...@apache.org.
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 72ed28034759c9d6061ac904cbd6365249fae9b3
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Mon Jun 20 11:41:28 2022 +0200

    (chores) camel-test-spring-junit5: use format specifiers in log messages
---
 .../java/org/apache/camel/test/spring/junit5/RouteCoverageDumper.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/components/camel-test/camel-test-spring-junit5/src/main/java/org/apache/camel/test/spring/junit5/RouteCoverageDumper.java b/components/camel-test/camel-test-spring-junit5/src/main/java/org/apache/camel/test/spring/junit5/RouteCoverageDumper.java
index 3bf5597dda7..76217c57bfd 100644
--- a/components/camel-test/camel-test-spring-junit5/src/main/java/org/apache/camel/test/spring/junit5/RouteCoverageDumper.java
+++ b/components/camel-test/camel-test-spring-junit5/src/main/java/org/apache/camel/test/spring/junit5/RouteCoverageDumper.java
@@ -62,7 +62,7 @@ public final class RouteCoverageDumper {
                 // ensure dir exists
                 boolean result = file.mkdirs();
                 if (!result) {
-                    LOG.error("mkdirs() failed for " + file);
+                    LOG.error("mkdirs() failed for {}", file);
                 }
                 file = new File(dir, name);
 


[camel] 08/08: (chores) camel-console: use format specifiers in log messages

Posted by or...@apache.org.
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 319ce436572a4c78107fd46f95eb07945e4c1558
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Mon Jun 20 11:44:43 2022 +0200

    (chores) camel-console: use format specifiers in log messages
---
 .../java/org/apache/camel/impl/console/DefaultDevConsolesLoader.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/core/camel-console/src/main/java/org/apache/camel/impl/console/DefaultDevConsolesLoader.java b/core/camel-console/src/main/java/org/apache/camel/impl/console/DefaultDevConsolesLoader.java
index da2c848e9d8..95f9c8245ac 100644
--- a/core/camel-console/src/main/java/org/apache/camel/impl/console/DefaultDevConsolesLoader.java
+++ b/core/camel-console/src/main/java/org/apache/camel/impl/console/DefaultDevConsolesLoader.java
@@ -70,8 +70,8 @@ public class DefaultDevConsolesLoader {
                 }
             }
         } catch (Exception e) {
-            LOG.warn("Error during scanning for custom dev-consoles on classpath due to: " + e.getMessage()
-                     + ". This exception is ignored.");
+            LOG.warn("Error during scanning for custom dev-consoles on classpath due to: {}. This exception is ignored.",
+                    e.getMessage());
         }
 
         return answer;


[camel] 04/08: (chores) camel-mock: remove unnecessary toString()

Posted by or...@apache.org.
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 3dc7d795d68f3c6f77c273c8151af6d8c9a2cd70
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Mon Jun 20 11:39:20 2022 +0200

    (chores) camel-mock: remove unnecessary toString()
---
 .../src/main/java/org/apache/camel/component/mock/TimeClause.java       | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/components/camel-mock/src/main/java/org/apache/camel/component/mock/TimeClause.java b/components/camel-mock/src/main/java/org/apache/camel/component/mock/TimeClause.java
index 03a3f9d22ce..8d8d295b703 100644
--- a/components/camel-mock/src/main/java/org/apache/camel/component/mock/TimeClause.java
+++ b/components/camel-mock/src/main/java/org/apache/camel/component/mock/TimeClause.java
@@ -197,7 +197,7 @@ public class TimeClause implements BinaryPredicate {
         }
 
         if (LOG.isDebugEnabled()) {
-            LOG.debug("Evaluated time clause [{}] with current: {}, other: {} -> {}", toString(), currentDate, otherDate,
+            LOG.debug("Evaluated time clause [{}] with current: {}, other: {} -> {}", this, currentDate, otherDate,
                     answer);
         }