-
Notifications
You must be signed in to change notification settings - Fork 108
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
Question & suggestion #5
Comments
Hi Tim, Currently we are not working on an async version, but we add your feature request to our backlog for future consideration. Thanks for the suggested option and pull request for not deleting S3 objects in DeleteMessage call. We will review the submission. |
👍 would prefer async S3 upload which triggers sending the SQS message via the handler |
Same here. I am working on a spring-cloud-aws application. Spring-cloud-aws uses the async client. |
Any updates? |
|
Hello. |
Here is a PR to add async to payload offloading: awslabs/payload-offloading-java-common-lib-for-aws#19 . I need reviewers, and not sure how to get anyone's attention in that repo. That support has a set of UTs along with it, and is fully consistent with the current design. I also created a draft PR in this repo that is based on the async payload offloading change: #79 . This also has a set of UTs, and is fully consistent with the current design. It is also fully asynchronous and non-blocking. If the payload offloading async support is approved, we can move the other PR in this repo to ready state and work on it. Anyone who is familiar with this repo, please let me know how I can kickstart this process. |
Is there any plan to bring the extended client in line with the standard client and produce an async version (or is this not possible due to the interactions with s3)?
It would be helpful if I could configure the client not to delete the s3 component of the message on the message delete action. This would then leave me an audit log of all the messages I'd received and I could then use the life-cycle attribute of the bucket to clear out the older messages that I no longer wish to retain.
Thanks,
Tim
The text was updated successfully, but these errors were encountered: