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

Fixed consume pending message calculation #626

Merged
merged 2 commits into from
Sep 11, 2024
Merged

Conversation

mtmk
Copy link
Collaborator

@mtmk mtmk commented Sep 10, 2024

Implemented a new method Delivered to accurately adjust the pending message size on message delivery. Ensured pending checks and size adjustments are correctly handled during reconnect and message delivery to maintain a consistent consumer state. Also updated tests to validate these changes.

resolves #608

Implemented a new method `Delivered` to accurately adjust the pending message
size on message delivery. Ensured pending checks and size adjustments are
correctly handled during reconnect and message delivery to maintain a
consistent consumer state. Also updated tests to validate these changes.
Copy link
Collaborator

@caleblloyd caleblloyd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mtmk mtmk merged commit 7b6af12 into main Sep 11, 2024
13 checks passed
@mtmk mtmk deleted the fix-consume-backpressure branch September 11, 2024 09:10
@mtmk mtmk mentioned this pull request Sep 11, 2024
divyeshio pushed a commit to divyeshio/nats.net that referenced this pull request Sep 13, 2024
* Fixed consume pending message calculation

Implemented a new method `Delivered` to accurately adjust the pending message
size on message delivery. Ensured pending checks and size adjustments are
correctly handled during reconnect and message delivery to maintain a
consistent consumer state. Also updated tests to validate these changes.

* dotnet format
mtmk added a commit that referenced this pull request Sep 17, 2024
* Fixed consume pending message calculation (#626)
* ServiceProvider callback for NATS DI configuration (#619)
* Nats web socket opts improvements (#623)
* Fix various disposable issues (#625)
* Make NuidWriter public (#618)
* NatsOpts.ConfigureWebSocketOpts callback handler (#605)
* Simplified NATS client (#607)
* Update docs (#595)
* Add default timeout to initial commands (#594)
* Extensive logging for reconnect debugging (#593)
* Add clear next step navigation to API index doc (#592)
* Add NATS client implementation (#589)
@mtmk mtmk mentioned this pull request Sep 17, 2024
mtmk added a commit that referenced this pull request Sep 17, 2024
* Fixed consume pending message calculation (#626)
* ServiceProvider callback for NATS DI configuration (#619)
* Nats web socket opts improvements (#623)
* Fix various disposable issues (#625)
* Make NuidWriter public (#618)
* NatsOpts.ConfigureWebSocketOpts callback handler (#605)
* Simplified NATS client (#607)
* Update docs (#595)
* Add default timeout to initial commands (#594)
* Extensive logging for reconnect debugging (#593)
* Add clear next step navigation to API index doc (#592)
* Add NATS client implementation (#589)
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.

MaxMsg=1 not respected. second message ALWAYS gets redelivered as if its ackwait started before yielded
2 participants