-
Notifications
You must be signed in to change notification settings - Fork 5.8k
Java V2: migrate v1 sqs example that compares single messaging to batch messaging #7509
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
Conversation
Why did you assign this back to me? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PE looks good,.
@tkhill-AWS everything looks good but there is a metadata validation failure. Once that's fixed I can merge it. |
@rlhagerm - I fixed the metadata validation error, but still see a WRITEME validation error, but I can't reproduce on my fork/branch. There, the WRITEME validation passes. For good measure, I ran the python |
…Consumer.java</emphasis> - Uses automatic batching of messages.' is not a valid string: it must start with an uppercase letter.
a92a3a6
to
6bb9452
Compare
This pull request migrates the hard-code v1 example that exists in the SQS Developer guide.Since Java V2 also offers the drop-in replacementSqsAsyncBatchManager
for theSqsAsyncClient
, which offers automatic batching, this approach was also highlighted in the example.These code example demonstrate two approaches to working with SQS messages in batches using the Java SDK. This PR combines the code examples from this original PR and this PR. It turned out that the two PRs were worked on concurrently without knowledge of the other. It made sense to combine them into one PR. The other PR can be canceled at this point.
CDD build is here.CDD build is here.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.