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:55 UTC
[camel] 02/02: (chores) camel-aws2-s3: use log guards
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;