-
Notifications
You must be signed in to change notification settings - Fork 621
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
Add GetNextPublishSeqNo() for Channel in confirm mode #478
base: master
Are you sure you want to change the base?
Add GetNextPublishSeqNo() for Channel in confirm mode #478
Conversation
👍 |
When merging it. #478 |
7f2fa64
to
8bcd255
Compare
8bcd255
to
dc3c53f
Compare
Hey folks, I'm posting this on behalf of the core team. As you have noticed, this client hasn't seen a lot of activity recently. Because this client has a long tradition of "no breaking public API changes", certain We would like to thank @streadway Team RabbitMQ has adopted a "hard fork" of this client What do we mean by "hard fork" and what does it mean for you? The entire history of the project What does change is that this new fork will accept reasonable breaking API changes according If your PR hasn't been accepted or reviewed, you are welcome to re-submit it for Note that it is a high season for holidays in some parts of the world, so we may be slower Thank you for using RabbitMQ and contributing to this client. On behalf of the RabbitMQ core team, |
For everyone who reaches this thread now, note that the exact same commit was released in |
GetNextPublishSeqNo
method onChannel
will be very useful for handling of especailly NACKed messages to keep track of message going to be published so that those can be republished or acted upon. Theconfirms.published
counter is already there and this method will just return the same counter.Resolves #476