Re: [PR] API, AWS: Retry S3InputStream reads [iceberg]

2024-09-24 Thread via GitHub
amogh-jahagirdar merged PR #10433: URL: https://github.com/apache/iceberg/pull/10433 -- 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: issues-unsubscr...@

Re: [PR] API, AWS: Retry S3InputStream reads [iceberg]

2024-09-24 Thread via GitHub
amogh-jahagirdar commented on PR #10433: URL: https://github.com/apache/iceberg/pull/10433#issuecomment-2371767216 Thanks for the reviews @danielcweeks! Merging. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] API, AWS: Retry S3InputStream reads [iceberg]

2024-09-23 Thread via GitHub
amogh-jahagirdar commented on code in PR #10433: URL: https://github.com/apache/iceberg/pull/10433#discussion_r1771831100 ## aws/src/test/java/org/apache/iceberg/aws/s3/TestS3InputStream.java: ## @@ -52,18 +52,15 @@ public void before() { createBucket("bucket"); } - @

Re: [PR] API, AWS: Retry S3InputStream reads [iceberg]

2024-09-23 Thread via GitHub
amogh-jahagirdar commented on code in PR #10433: URL: https://github.com/apache/iceberg/pull/10433#discussion_r1771783667 ## core/src/main/java/org/apache/iceberg/io/RetryableInputStream.java: ## @@ -0,0 +1,130 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under on

Re: [PR] API, AWS: Retry S3InputStream reads [iceberg]

2024-09-19 Thread via GitHub
SandeepSinghGahir commented on PR #10433: URL: https://github.com/apache/iceberg/pull/10433#issuecomment-2361969290 @amogh-jahagirdar Any tentative timeline on merging of this PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitH