这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@jbret89
Copy link
Contributor

@jbret89 jbret89 commented Apr 23, 2025

This pull request introduces a new feature to support SQS visibility heartbeat in the machinery project. The main changes include:

  • New Configuration Option: Added VisibilityHeartBeat to enable/disable the visibility heartbeat.
  • Mock Implementation: Created a mock for sqsiface.SQSAPI using testify/mock.
  • Unit Tests: Added tests to assert the number of calls to ChangeMessageVisibility.
  • Code Refactoring: Refactored code to integrate the new feature.

Changes

  • Added VisibilityHeartBeat configuration option.
  • Implemented MockSQSAPI for testing.
  • Updated unit tests.
  • Refactored codebase.

Testing

  • Verified ChangeMessageVisibility method calls.
  • Ensured all existing tests pass.

Documentation

  • Updated configuration documentation to include VisibilityHeartBeat.

This feature aims to improve message processing reliability by periodically extending the visibility timeout of in-flight messages, preventing other consumers from processing the same message simultaneously and avoiding duplicate consumption.

@jbret89 jbret89 changed the title Add sqs visibility hearth beat Add AWS SQS Visibility Heartbeat Feature Apr 23, 2025
@jbret89 jbret89 marked this pull request as ready for review April 23, 2025 11:02
@jbret89 jbret89 mentioned this pull request Apr 23, 2025
@RichardKnop RichardKnop merged commit dfd7622 into RichardKnop:master Jul 19, 2025
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.

2 participants