Skip to content

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

Merged
merged 14 commits into from
Jul 17, 2025

Conversation

tkhill-AWS
Copy link
Collaborator

@tkhill-AWS tkhill-AWS commented Jul 2, 2025

This pull request migrates the hard-code v1 example that exists in the SQS Developer guide.

Since Java V2 also offers the drop-in replacement SqsAsyncBatchManager for the SqsAsyncClient, 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.

@github-actions github-actions bot added the Java-v2 This issue relates to the AWS SDK for Java V2 label Jul 2, 2025
@tkhill-AWS tkhill-AWS requested a review from scmacdon July 2, 2025 14:51
@tkhill-AWS
Copy link
Collaborator Author

Why did you assign this back to me?

Copy link
Contributor

@scmacdon scmacdon left a 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,.

@scmacdon scmacdon added On Call Review needed This work needs an on-call review Task A general update to the code base for language clarification, missing actions, tests, etc. labels Jul 10, 2025
@rlhagerm
Copy link
Contributor

@tkhill-AWS everything looks good but there is a metadata validation failure. Once that's fixed I can merge it.

@rlhagerm rlhagerm removed the On Call Review needed This work needs an on-call review label Jul 11, 2025
@tkhill-AWS
Copy link
Collaborator Author

tkhill-AWS commented Jul 14, 2025

@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 writeme script on my fork/branch again, but no files were changed.

@beqqrry-aws beqqrry-aws added the On Call Review complete On call review complete label Jul 17, 2025
@beqqrry-aws beqqrry-aws merged commit 6cf888b into awsdocs:main Jul 17, 2025
17 of 18 checks passed
@tkhill-AWS tkhill-AWS deleted the java_sqs_batch branch July 21, 2025 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Java-v2 This issue relates to the AWS SDK for Java V2 On Call Review complete On call review complete Task A general update to the code base for language clarification, missing actions, tests, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants