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

Redelivery count field addition to message #158

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jDzames
Copy link

@jDzames jDzames commented Dec 11, 2020

Fix for #155
As this is supported by server, just need to add field for redeliveryCount. Tested locally within my project.

@liqweed
Copy link

liqweed commented Feb 7, 2021

I'm not sure the build failures on this PR are relate to the change.
Can anyone please review it?

@lecodeski
Copy link

bump – any news on this?
is there maybe something where I could support?

@jDzames
Copy link
Author

jDzames commented Apr 20, 2021

I'm not sure the build failures on this PR are relate to the change.
Can anyone please review it?

Should only be due to version. Otherwise it works, I have tried it locally...
I let version for repo owners to set it as they wish before releasing.

@lecodeski
Copy link

ping again…
Would it be possible to bump the version number? We would actually need this feature.

But aside from from just the version number – any chance this also gets released soon?

@Yamaha32088
Copy link

+1 Would be nice to have. Could someone please merge this or take a look and provide feedback?

@joriatyBen
Copy link

@Yamaha32088 Since @blanorama and me waited our asses of we decided to move on and changed our project lib to nats jetstream which is packed with a bunch of useful features including redelivery count. Due to the fact that nats streaming is marked with a deprecated sign and the java stan repo is not maintained since 2019 this should be your solution too!

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.

5 participants