Skip to content

add redelivered attribute to message object - #405

Closed
nffdiogosilva wants to merge 1 commit into
Bogdanp:masterfrom
nffdiogosilva:feature/rabbitmq-redelivered
Closed

nffdiogosilva wants to merge 1 commit into
Bogdanp:masterfrom
nffdiogosilva:feature/rabbitmq-redelivered

Conversation

@nffdiogosilva

Copy link
Copy Markdown

I think something like this would fix issue #404

@and3rson

and3rson commented Oct 11, 2021 •

Copy link
Copy Markdown

+1, this is exactly the missing functionality that I was looking for.

@Bogdanp Can we have this? I just found this PR while looking for a way to know if the previous delivery failed to be able to stop my pipeline, since the previous delivery might have left the database in an inconsistent state due to worker crash.

TL;DR: looking for a way to know if message was redelivered (worker crashed, etc) in order to gracefully terminate my pipeline.

@Bogdanp

Bogdanp commented Oct 12, 2021

Copy link
Copy Markdown
Owner

Yes, I think this would be fine to merge. I'll probably do it this weekend.

@Bogdanp

Bogdanp commented Oct 17, 2021

Copy link
Copy Markdown
Owner

Thanks! This is now merged w/ rebase.

@Bogdanp Bogdanp closed this Oct 17, 2021
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.

3 participants