-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
feat: migrate to async_nats
client
#15533
Conversation
✅ Deploy Preview for vrl-playground ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
✅ Deploy Preview for vector-project ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
Regression Test Results
Run ID: fa37f44d-fd16-4727-ad0e-c18288530361 Explanation
A regression test is an integrated performance test for
The table below, if present, lists those experiments that have experienced a
statistically significant change in their No interesting changes in Fine details of change detection per experiment.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice. Thanks for doing this.
The issue you mentioned around buffering should be fine, from what I can tell the async library isn't doing any buffering so it shouldn't be a problem.
There are a couple of CI errors that need fixing. The integration tests have some compile errors. nats.rs
needs to be formatted correctly, which you can do by running the file through rustfmt
.
Regression Test Results
Run ID: 6e988203-128f-4e9c-bba8-b162ec5e6d2c Explanation
A regression test is an integrated performance test for
The table below, if present, lists those experiments that have experienced a
statistically significant change in their No interesting changes in Fine details of change detection per experiment.
|
Hi, I'm getting a bunch of integration test errors now, for example:
Are they passing locally for you? You can run the tests by running Unfortunately the integration tests weren't set to run in CI for this PR. I've just enabled it so we should now see the results on each push. |
Didn't mean to close this. |
Since this PR hasn't received any updates I'll try continuing it in #18165 |
That would be great, thank you! |
Implements migration from
nats::asynk
client to more modern official clientasync_nats
Closes #15534