Add TCP_NODELAY

[denyaalt]

Hi,
Is it possible to add the TCP_NODELAY setting to SO_REUSEPORT to use them together?
Thanks.

[jokester]

Hi @denyaalt . I didn't have a plan by far, because I don't know what it does.

Can you brief me about, in what case and why would people want it?

[denyaalt]

TCP_NODELAY disables the Nagle algorithm (disables buffering and delays), which allows data to be sent as quickly as possible.

[jokester]

@denyaalt Thanks for the explanation, didn't know that.

I did a research about TCP_NODELAY in nodejs. Maybe node.js already have that option around v18 as default in socket and http servers, via PR.

Sorry but can you check that too? (If node.js already have that capability I tend to not do it again)

[denyaalt]

Yes, this option is available in Node.js when we directly open a TCP socket, I'm not sure if it will work after your wrapper.
My colleague reported that when opening SO_REUSEPORT sockets from systemd, the TCP_NODELAY option does not work from Node.js, the user needs to additionally set the TCP_NODELAY flag in systemd.

[jokester]

@denyaalt that sounds more like a bug in node.js (unlike SO_REUSEPORT they decided to not support) 🤔

Or, you are welcome to fork this repo and create a local build with TCP_NODELAY and see if it helps in your situation.