8000 NetworkTarget with Socket.NoDelay to disable TCP_NODELAY · Issue #5778 · NLog/NLog · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

NetworkTarget with Socket.NoDelay to disable TCP_NODELAY #5778

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
Closed
snakefoot opened this issue Apr 12, 2025 · 1 comment · Fixed by #5779
Closed

NetworkTarget with Socket.NoDelay to disable TCP_NODELAY #5778

snakefoot opened this issue Apr 12, 2025 · 1 comment · Fixed by #5779
Labels
feature network/NLogViewer-target
Milestone
6.0

Comments

@snakefoot
Copy link
Contributor
snakefoot commented Apr 12, 2025 •
edited
Loading

TCP_NODELAY disables the nagle-algorithm and delayed ACK, which removes a delay of 200ms for the first message to reduce ACKS.

Nagle-algorithm is important for high-latency / low-bandwith network-connections, but now most have fiber-connection or better.

Believe most people will not notice the overhead of extra ACKS, but will notice the 200ms delay.

Socket.NoDelay should be assigned to true by default, but should of course be configurable.

The feature should also be implemented for NLog v5.5 but with default value false.
@snakefoot snakefoot added this to the 6.0 milestone Apr 12, 2025
@snakefoot
Copy link
Contributor Author
snakefoot commented Apr 12, 2025 •
edited
Loading

Bonus task is also to set SendTimeoutSeconds default-value to 100 secs for NLog v6.

This was referenced Apr 12, 2025
Merged
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature network/NLogViewer-target
Projects
None yet
Milestone
6.0
Development

Successfully merging a pull request may close this issue.

NetworkTarget - Introduced option NoDelay to disable delayed ACK snakefoot/NLog
1 participant
@snakefoot
0