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

[Test] Disable Nagle's algorithm #4390

Open

c410-f3r wants to merge 1 commit into diesel-rs:master from c410-f3r:patch-1

Contributor

c410-f3r commented Dec 15, 2024 •

edited

Loading

Much to my surprise, almost all database clients apply stream.set_nodelay(true) by default.

mysql: https://github.com/blackbeam/rust-mysql-simple/blob/bdc32fae7f2869c29ca52284dff7bf19beeaca56/src/io/tcp.rs#L102
sqlx: https://github.com/launchbadge/sqlx/blob/1678b19a4672fd6a18b4891c53bf0b57638b92a4/sqlx-core/src/net/socket/mod.rs#L201
tokio-postgres: https://github.com/sfackler/rust-postgres/blob/6ae17e0f2d174fc4d5592ac7869909d7bdd346d9/tokio-postgres/src/connect_socket.rs#L27

If possible, I would like to see the impact of this parameter in the wtx project.


          Update wtx.rs

f542c32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet