Skip to content
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

Monitor for progress and optionally shutdown (leave consumer group), skip message or send to DLQ #34

Closed
astubbs opened this issue Nov 26, 2020 · 2 comments
Labels
Milestone

Comments

@astubbs
Copy link
Contributor

astubbs commented Nov 26, 2020

Some issues could cause a lack of any progress along a partition. For example infinite retries on a message to an external system that will always be rejected. Or a possible bug in the handling code causing the message to not be retried.
If a situation like this is found, there should be options to either DLQ the message and move on, or potentially shutdown and leave the consumer group.
User code should be able to return either a fatal error type or a retriable error type.

The system at the moment does do mutual thread supervision for the two long running threads (control and poller) for thread death, but this issue is about adding an overall offset level fail safe, which will include monitoring for stuck messages that can't ever succeed. It is yet to be decided what options there will be in this case, for example: shutdown, dead letter queue, skip. Skip and shutdown would be easy to implement so they'll be first.

Note that to skip a message, a user can already simply return from their processing function without throwing an exception, and potentially without processing it.

@astubbs astubbs added enhancement New feature or request ver:0.3.1 high labels Dec 14, 2020
@astubbs astubbs added this to the 0.3.1 milestone Dec 14, 2020
@astubbs astubbs mentioned this issue Jan 14, 2021
5 tasks
astubbs added a commit to astubbs/parallel-consumer that referenced this issue Jan 15, 2021
@astubbs
Copy link
Contributor Author

astubbs commented Jan 26, 2021

Related: #71 Health-checks

@johnbyrnejb
Copy link
Contributor

Closing Issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants