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.

Sign up for GitHub

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

Jump to bottom

[FIXED] JetStream flow control may stall in some conditions #837

Merged
merged 3 commits into from Oct 5, 2021
Merged

[FIXED] JetStream flow control may stall in some conditions #837

merged 3 commits into from Oct 5, 2021

Commits on Oct 5, 2021

  1. [FIXED] JetStream flow control may stall in some conditions 

    Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
    @kozlovic
    kozlovic committed Oct 5, 2021
    Copy the full SHA
    66ba98e View commit details
    Browse the repository at this point in the history

  2. Fixed ordered consumer test 

    Several issues:
- We send a "EOF" empty message which should not be counted as
the number of chunks used to reconstitute the asset
- Some "message filters" that are removed as part of the execution
of the filter's callback would not be put back for the following
"sync" test (we test async then sync).

Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
    @kozlovic
    kozlovic committed Oct 5, 2021
    Copy the full SHA
    aedc479 View commit details
    Browse the repository at this point in the history

  3. Updates based on code review 

    Signed-off-by: Ivan Kozlovic <ivan@synadia.com>
    @kozlovic
    kozlovic committed Oct 5, 2021
    Copy the full SHA
    c564a49 View commit details
    Browse the repository at this point in the history