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

lakshsingla pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git


The following commit(s) were added to refs/heads/master by this push:
     new 5e603ac5ff1 Adding more logging for s3 RetryableS3OutputStream (#16117)
5e603ac5ff1 is described below

commit 5e603ac5ff1a6fd0fc8fc2a68701d6922d2edee3
Author: Karan Kumar <[email protected]>
AuthorDate: Thu Mar 14 11:35:57 2024 +0530

    Adding more logging for s3 RetryableS3OutputStream (#16117)
    
    Adding more logging for s3 RetryableS3OutputStream which would help us 
determine if the chunk size needs to be adjusted.
---
 .../apache/druid/storage/s3/output/RetryableS3OutputStream.java    | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git 
a/extensions-core/s3-extensions/src/main/java/org/apache/druid/storage/s3/output/RetryableS3OutputStream.java
 
b/extensions-core/s3-extensions/src/main/java/org/apache/druid/storage/s3/output/RetryableS3OutputStream.java
index 82f5cd812cc..c71bb4e788b 100644
--- 
a/extensions-core/s3-extensions/src/main/java/org/apache/druid/storage/s3/output/RetryableS3OutputStream.java
+++ 
b/extensions-core/s3-extensions/src/main/java/org/apache/druid/storage/s3/output/RetryableS3OutputStream.java
@@ -269,7 +269,12 @@ public class RetryableS3OutputStream extends OutputStream
     // Closeables are closed in LIFO order
     closer.register(() -> {
       // This should be emitted as a metric
-      LOG.info("Total push time: [%d] ms", 
pushStopwatch.elapsed(TimeUnit.MILLISECONDS));
+      LOG.info(
+          "Pushed total [%d] parts containing [%d] bytes in [%d]ms.",
+          numChunksPushed,
+          resultsSize,
+          pushStopwatch.elapsed(TimeUnit.MILLISECONDS)
+      );
     });
 
     closer.register(() -> 
org.apache.commons.io.FileUtils.forceDelete(chunkStorePath));


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to