Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support v2 JS.ACK format #351

Merged
merged 8 commits into from
Sep 11, 2022
Merged

Support v2 JS.ACK format #351

merged 8 commits into from
Sep 11, 2022

Conversation

bruth
Copy link
Member

@bruth bruth commented Sep 10, 2022

See nats-io/nats-and-architecture-design#49

Signed-off-by: Byron Ruth <b@devel.io>
Signed-off-by: Byron Ruth <b@devel.io>
Signed-off-by: Byron Ruth <b@devel.io>
Signed-off-by: Byron Ruth <b@devel.io>
Signed-off-by: Byron Ruth <b@devel.io>
Signed-off-by: Byron Ruth <b@devel.io>
@bruth
Copy link
Member Author

bruth commented Sep 10, 2022

The CI reports one failure on the branch for 3.9/uvloop, but the PR passed. Do they run the same tests? Was the one that failed just flaky?

@wallyqs
Copy link
Member

wallyqs commented Sep 10, 2022

Thanks Byron! Yes there a couple of flaky tests under uvloop CI run...

Signed-off-by: Waldemar Quevedo <wally@nats.io>
Signed-off-by: Waldemar Quevedo <wally@nats.io>
@wallyqs wallyqs merged commit acbf469 into main Sep 11, 2022
@wallyqs wallyqs deleted the v2-token-count branch September 11, 2022 05:37
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants