Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add the capability to speed up S3 uploads using AWS transfer manager #17669

Open
suraj-goel opened this issue Jan 27, 2025 · 3 comments · May be fixed by #17674
Open

Add the capability to speed up S3 uploads using AWS transfer manager #17669

suraj-goel opened this issue Jan 27, 2025 · 3 comments · May be fixed by #17674

Comments

@suraj-goel
Copy link
Contributor

suraj-goel commented Jan 27, 2025

Description

Motivation

  • To improve speed of uploads to S3.
@suraj-goel
Copy link
Contributor Author

I will work on this issue.

@LakshSingla
Copy link
Contributor

I was working on something similar a while back. Maybe these changes will help you #14834. What I found was that the speed improvements weren't a lot for MSQ because transfer manager in the SDK v1(?) didn't parallelize if the downloads were chunked. It helped a bit when segments were getting downloaded and uploaded. However, for some reason, segments uploaded without the transfer manager were slowly downloaded when downloaded with the transfer manager.
As such, it would be really cool if you can benchmark the performance of the changes, along with speedups/slowdown when downloaded and uploaded pre-post your patch.

@suraj-goel suraj-goel linked a pull request Jan 28, 2025 that will close this issue
10 tasks
@suraj-goel
Copy link
Contributor Author

I was working on something similar a while back. Maybe these changes will help you #14834. What I found was that the speed improvements weren't a lot for MSQ because transfer manager in the SDK v1(?) didn't parallelize if the downloads were chunked. It helped a bit when segments were getting downloaded and uploaded. However, for some reason, segments uploaded without the transfer manager were slowly downloaded when downloaded with the transfer manager. As such, it would be really cool if you can benchmark the performance of the changes, along with speedups/slowdown when downloaded and uploaded pre-post your patch.

@LakshSingla, Thanks for the suggestions.
Right now, My changes are focussed on optimizing the uploads.
I will try to come up with some existing benchmarks or create a new one for transfer manager.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants