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/10/24 06:56:53 UTC

[camel] branch main updated (814356b9e31 -> 9c932dcb450)

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 814356b9e31 CAMEL-18630: camel-jbang - Add get health command
     new c9090285dad (chores) camel-as2-api: use log guards
     new 9c932dcb450 (chores) camel-aws2-s3: use log guards

The 2 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:
 .../org/apache/camel/component/as2/api/protocol/ResponseMDN.java  | 4 +++-
 .../component/aws2/s3/stream/AWS2S3StreamUploadProducer.java      | 8 ++++++--
 2 files changed, 9 insertions(+), 3 deletions(-)


[camel] 02/02: (chores) camel-aws2-s3: use log guards

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 9c932dcb450f78cdaecb6af67dda5c298c27143f
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Mon Oct 24 06:34:55 2022 +0200

    (chores) camel-aws2-s3: use log guards
    
    Prevent extra overhead due to converting an AtomicInteger to string
---
 .../component/aws2/s3/stream/AWS2S3StreamUploadProducer.java      | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/components/camel-aws/camel-aws2-s3/src/main/java/org/apache/camel/component/aws2/s3/stream/AWS2S3StreamUploadProducer.java b/components/camel-aws/camel-aws2-s3/src/main/java/org/apache/camel/component/aws2/s3/stream/AWS2S3StreamUploadProducer.java
index c57f6d4b0e7..328d8b182d9 100644
--- a/components/camel-aws/camel-aws2-s3/src/main/java/org/apache/camel/component/aws2/s3/stream/AWS2S3StreamUploadProducer.java
+++ b/components/camel-aws/camel-aws2-s3/src/main/java/org/apache/camel/component/aws2/s3/stream/AWS2S3StreamUploadProducer.java
@@ -207,8 +207,12 @@ public class AWS2S3StreamUploadProducer extends DefaultProducer {
                         .build();
 
         uploadResult = getEndpoint().getS3Client().completeMultipartUpload(compRequest);
-        LOG.info("Completed upload for the part {} with etag {} at index {}", part, uploadResult.eTag(),
-                index);
+
+        // Converting the index to String can cause extra overhead
+        if (LOG.isInfoEnabled()) {
+            LOG.info("Completed upload for the part {} with etag {} at index {}", part, uploadResult.eTag(),
+                    index);
+        }
 
         index.getAndSet(0);
         initResponse = null;


[camel] 01/02: (chores) camel-as2-api: use log guards

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 c9090285dad3c2cf6d83a1f3e2b5356161a6fbd4
Author: Otavio Rodolfo Piske <an...@gmail.com>
AuthorDate: Mon Oct 24 06:34:24 2022 +0200

    (chores) camel-as2-api: use log guards
    
    Prevent extra overhead due to parsing and conversion
---
 .../java/org/apache/camel/component/as2/api/protocol/ResponseMDN.java | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/ResponseMDN.java b/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/ResponseMDN.java
index ab1a32ba817..1cb577e25a0 100644
--- a/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/ResponseMDN.java
+++ b/components/camel-as2/camel-as2-api/src/main/java/org/apache/camel/component/as2/api/protocol/ResponseMDN.java
@@ -243,7 +243,9 @@ public class ResponseMDN implements HttpResponseInterceptor {
             }
         }
 
-        LOG.debug(AS2Utils.printMessage(response));
+        if (LOG.isDebugEnabled()) {
+            LOG.debug(AS2Utils.printMessage(response));
+        }
     }
 
     private String createMdnDescription(