-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
sqs consumers #33659
sqs consumers #33659
Conversation
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
…ng, refactor in sqs benchmark
For what I can see I think the real game change is your #33658 This PR could anyway help to scale out an sqs+s3 input on a single agent instance, giving the option to basically having a similar behaviour (specifically joint with your above PR) to set up different repeated single inputs for the same sqs queue nit: I found the benchmark a little unclear, I've tried to make it more readable and effective (no matter how many msgs etc the constant receive was always returning 1 message) I still have to add docs and default settings for |
This pull request is now in conflicts. Could you fix it? 🙏
|
This pull request is now in conflicts. Could you fix it? 🙏
|
1 similar comment
This pull request is now in conflicts. Could you fix it? 🙏
|
Closing this as it is for now outdated and hasn't been worked on for quite some time. Feel free to reopen it if need be. |
Enhancement
What does this PR do?
Why is it important?
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Author's Checklist
How to test this PR locally
Related issues
Use cases
Screenshots
Logs