Skip to content

Conversation

@fuatbasik
Copy link

Context
We have integrated analytics-accelerator-s3 into Iceberg in PR #12299. Currently, Iceberg customers need to enable the s3.analytics-accelerator.enabled flag in S3FileIOProperties to use the library. There is a proposal to make Analytics stream the default input stream: https://docs.google.com/document/d/13shy0RWotwfWC_qQksb95PXdi-vSUCKQyDzjoExQEN0

Description of Change
Starting from 1.2.2 AAL will allow consumers to pass Retry strategy to execute on Input Stream from S3 to AAL. (awslabs/analytics-accelerator-s3#340)

With this change we are using that stream to ensure parity with S3SeekableInputStream and Analytics Stream on Iceberg.

Unlike current retries where stream has to be re-opened from the last-read position, in these retries, we do not need to do anything on the stream level as AAL will ensure reads are idempotent.

Testing
Extended FlakyInputStream tests to AAL and confirmed all tests are passing in the presence of exceptions and non-retriable exceptions are not retries.

@github-actions
Copy link

github-actions bot commented Sep 5, 2025

This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the [email protected] list. Thank you for your contributions.

@github-actions github-actions bot added the stale label Sep 5, 2025
@github-actions
Copy link

This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time.

@github-actions github-actions bot closed this Sep 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants