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

perf: Make KCP the default transport #398

Merged
merged 1 commit into from
Oct 16, 2020
Merged

perf: Make KCP the default transport #398

merged 1 commit into from
Oct 16, 2020

Conversation

paulpach
Copy link
Contributor

KCP transport is a better default than TCP trasport

@sonarcloud
Copy link

sonarcloud bot commented Oct 15, 2020

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

100.0% 100.0% Coverage
0.0% 0.0% Duplication

@uweeby
Copy link
Member

uweeby commented Oct 15, 2020

Would it being default also entail changing over the test fixtures and examples?

@paulpach
Copy link
Contributor Author

Would it being default also entail changing over the test fixtures and examples?

yes, we should change them too, separate pr.

@paulpach paulpach merged commit dc6cc4f into master Oct 16, 2020
@paulpach paulpach deleted the kcpdefault branch October 16, 2020 16:42
github-actions bot pushed a commit that referenced this pull request Oct 16, 2020
## [49.3.2](v49.3.1...v49.3.2) (2020-10-16)

### Performance Improvements

* Make KCP the default transport ([#398](#398)) ([dc6cc4f](dc6cc4f))
@github-actions
Copy link
Contributor

🎉 This PR is included in version 49.3.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants