spring-cloud-aws icon indicating copy to clipboard operation
spring-cloud-aws copied to clipboard

Add BatchVisibility listener method parameter (#832)

Open clementdenis opened this issue 9 months ago • 0 comments

:loudspeaker: Type of change

  • [ ] Bugfix
  • [x] New feature
  • [ ] Enhancement
  • [ ] Refactoring

:scroll: Description

Implementation for #832

This is only a crude implementation (see TODO in QueueMessageVisibility) but should work good enough.

Note: there's an API change on the VisibilityHandlerMethodArgumentResolver constructor, but there shouldn't be any reason to use this class directly.

:bulb: Motivation and Context

See #832

:green_heart: How did you test it?

Through unit / integration tests

:pencil: Checklist

  • [x] I reviewed submitted code
  • [x] I added tests to verify changes
  • [x] I updated reference documentation to reflect the change
  • [x] All tests passing
  • [x] No breaking changes

:crystal_ball: Next steps

Maybe rework QueueMessage(Batch)Visibility to be more similar to acknowledgment design (using a callback).

clementdenis avatar May 02 '24 10:05 clementdenis