[GitHub] [camel] dk2k commented on a change in pull request #5842: fixes based on static analysis check with FindBugs

[GitBox <gi...@apache.org>]

dk2k commented on a change in pull request #5842:
URL: https://github.com/apache/camel/pull/5842#discussion_r672208153



##########
File path: components/camel-aws/camel-aws2-s3/src/main/java/org/apache/camel/component/aws2/s3/AWS2S3Producer.java
##########
@@ -214,7 +214,10 @@ public void processMultiPart(final Exchange exchange) throws Exception {
 
                 LOG.trace("Uploading part [{}] for {}", part, keyName);
                 try (InputStream fileInputStream = new FileInputStream(filePayload)) {
-                    fileInputStream.skip(filePosition);
+                    long skipped = fileInputStream.skip(filePosition);
+                    if (skipped == 0) {
+                        LOG.warn("0 bytes skipped");

Review comment:
       please elaborate on your requirement




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@camel.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org