On Thu, Oct 11, 2012 at 12:31 AM, Kazu Yamamoto
Hi Jeremy,
One question and one suggestion.
Question: you set NoDelay to the listen socket. What is the intention? Do connected sockets inherit this flag? Regardless of inheritance, I'm not sure this option improves the performance of HTTP servers. It is the option for interactive applications such as ssh.
I am pretty sure I just copied that flag from snap. Yay for cargo culting! It tests show that it doesn't help anything then we can remove it. Or at least add a comment that is doesn't appear to help anything..
Suggestion: allocating ByteString is a heavy job. If you can reuse the ByteString of recv() for multiple HTTP requests in an HTTP connection, the performance would be improved much.
How would that even be possible? - jeremy