jon-valliere commented on code in PR #44: URL: https://github.com/apache/mina/pull/44#discussion_r1597513918
########## mina-core/src/main/java/org/apache/mina/transport/socket/DefaultSocketSessionConfig.java: ########## @@ -37,7 +37,7 @@ public class DefaultSocketSessionConfig extends AbstractSocketSessionConfig { private static final int DEFAULT_SO_LINGER = -1; - private static final boolean DEFAULT_TCP_NO_DELAY = false; + private static final boolean DEFAULT_TCP_NO_DELAY = true; // Disable Nagle algorithm by default Review Comment: In my other IO framework, all of the TCP/SOCK options all have `unset` states where the framework code will not call the SOCK configuration at all and allow the OS defaults to be applied. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@mina.apache.org For additional commands, e-mail: dev-h...@mina.apache.org