Re: [PR] AWS: add retry logic to S3InputStream [iceberg]

2024-08-14 Thread via GitHub
steveloughran commented on code in PR #4912: URL: https://github.com/apache/iceberg/pull/4912#discussion_r1717062832 ## aws/src/main/java/org/apache/iceberg/aws/s3/S3InputStream.java: ## @@ -172,31 +274,62 @@ private void positionStream() throws IOException { } // cl

Re: [PR] AWS: add retry logic to S3InputStream [iceberg]

2024-06-12 Thread via GitHub
puchengy commented on PR #4912: URL: https://github.com/apache/iceberg/pull/4912#issuecomment-2163990331 @amogh-jahagirdar Hi, I am checking in to see if there is any update? thank you! -- This is an automated message from the Apache Git Service. To respond to the message, please log on t

Re: [PR] AWS: add retry logic to S3InputStream [iceberg]

2024-05-28 Thread via GitHub
amogh-jahagirdar commented on PR #4912: URL: https://github.com/apache/iceberg/pull/4912#issuecomment-2136044924 Hey @puchengy I'm wokring on it! I am aiming to have a PR out sometime this week to address this. -- This is an automated message from the Apache Git Service. To respond to th

Re: [PR] AWS: add retry logic to S3InputStream [iceberg]

2024-05-28 Thread via GitHub
puchengy commented on PR #4912: URL: https://github.com/apache/iceberg/pull/4912#issuecomment-2136041865 @amogh-jahagirdar Hello, i wonder is there any update on this? Our clients from time to time hit the issue and cause some interruptions to their workload. Thanks! -- This is an automa

Re: [PR] AWS: add retry logic to S3InputStream [iceberg]

2024-05-21 Thread via GitHub
puchengy commented on PR #4912: URL: https://github.com/apache/iceberg/pull/4912#issuecomment-2123007781 thank you @danielcweeks and @amogh-jahagirdar ! -- 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

Re: [PR] AWS: add retry logic to S3InputStream [iceberg]

2024-05-20 Thread via GitHub
puchengy commented on PR #4912: URL: https://github.com/apache/iceberg/pull/4912#issuecomment-2120650709 @jackye1995 @xiaoxuandev @danielcweeks @rdblue @amogh-jahagirdar @nastra appreciate your feedback on this, thank you. -- This is an automated message from the Apache Git Service. To re

Re: [PR] AWS: add retry logic to S3InputStream [iceberg]

2024-05-15 Thread via GitHub
puchengy commented on PR #4912: URL: https://github.com/apache/iceberg/pull/4912#issuecomment-2114092322 @jackye1995 Hi, I also encountered the same issue as described in https://github.com/apache/iceberg/issues/10340 can we continue the PR to address the issue here? ALSo I wonder have you