actix / actix/actix-web

Make a decision whether we should enable `TCP_NODELAY` by default or not

Đang mở
#3,917 4 bình luận 0 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
Rust
Star
24.8k
Fork
1.9k
Merge trung bình
23 giờ 10 phút
Pull request đã merge (30 ngày)
26

Mô tả

> The problem here should be related to Nagle’s algorithm, i.e. the `TCP_NODELAY` option.
> On Node.js, it's enabled by default: https://nodejs.org/api/http.html#httpcreateserveroptions-requestlistener
> On actix-web (actix-http), it's disabled by default (no config).
> I've tested on your benchmark and confirmed enabling it via `actix_tls` resolved the latency difference.
> Maybe we should make it enable by default, but it's a behavior change and should happen on major release.
> For now, I'm going to add a helper method to enable it easily.

_Originally posted by @JohnTitor in [#3841](https://github.com/actix/actix-web/issues/3841#issuecomment-3883169095)_

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Đánh giá

Issue này chưa được đánh giá.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.