ashangit commented on code in PR #3293:
URL: https://github.com/apache/celeborn/pull/3293#discussion_r2115544486


##########
multipart-uploader/multipart-uploader-s3/src/main/java/org/apache/celeborn/S3MultipartUploadHandler.java:
##########
@@ -66,6 +67,9 @@ public class S3MultipartUploadHandler implements 
MultipartUploadHandler {
 
   private Integer s3MultiplePartUploadMaxRetries;
 
+  long initBackoff = 100; // 100ms

Review Comment:
   Ok, but it means that we should also change the usage of the retry policy 
here 
https://github.com/apache/celeborn/blob/5f58fb1e3e0fecc721db4e38292b3f6d2624a1af/multipart-uploader/multipart-uploader-s3/src/main/java/org/apache/celeborn/S3MultipartUploadHandler.java#L85
 to ensure we set same config here than the one that will be set on the 
celeborn config



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to