Skip to content

feat: SQS Partial batch Utility #120

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 12 commits into from
Oct 5, 2020
Merged

feat: SQS Partial batch Utility #120

merged 12 commits into from
Oct 5, 2020

Conversation

pankajagrawal16
Copy link
Contributor

@pankajagrawal16 pankajagrawal16 commented Oct 3, 2020

Issue #, if available:

Description of changes:

Closes #119

Checklist

Breaking change checklist

RFC issue #:

  • Migration process documented
  • Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@pankajagrawal16 pankajagrawal16 force-pushed the partial-batch branch 8 times, most recently from a490adc to 2bf4d6d Compare October 4, 2020 06:35
Copy link
Contributor

@msailes msailes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@pankajagrawal16 pankajagrawal16 merged commit 6ebbd3f into master Oct 5, 2020
@pankajagrawal16 pankajagrawal16 deleted the partial-batch branch October 5, 2020 14:45
jeromevdl pushed a commit to jeromevdl/aws-lambda-powertools-java that referenced this pull request Oct 19, 2020
* Initial API skeleton for Partial SQS batch util

* Better error handling

* Initial Test cases setup and some refactorings

* Full Test cases coverage

* Rename API method for batch processing

* java docs

* public docs update

* Fix correct place holder for queuename and account

* Example usage with relevant permissions

* Minor doc updates

* Ranme method to set custom sqs client

* Make test less confusing

Co-authored-by: Pankaj Agrawal <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SQS partial batch failure
2 participants