Enabled TCP NoDelay by default

We now enable TCP NoDelay by default to avoid delayed TCP ACK's. Our flushing behavior isn't ideal yet, but combining messages allows to reduce the number of TCP packets.

[resolves #334][#138]

Author: mp911de

README.md

@@ -90,8 +90,8 @@ Mono<Connection> connectionMono = Mono.from(connectionFactory.create());
 | `sslCert`         | Path to SSL certificate for TLS authentication in PEM format. _(Optional)_
 | `sslPassword`     | Key password to decrypt SSL key. _(Optional)_
 | `sslHostnameVerifier` | `javax.net.ssl.HostnameVerifier` implementation. _(Optional)_
-| `tcpNoDelay`      | Enabled/disable TCP NoDelay. Disabled by default. _(Optional)_
-| `tcpKeepAlive`    | Enabled/disable TCP KeepAlive. Disabled by default. _(Optional)_
+| `tcpNoDelay`      | Enable/disable TCP NoDelay. Enabled by default. _(Optional)_
+| `tcpKeepAlive`    | Enable/disable TCP KeepAlive. Disabled by default. _(Optional)_
 
 **Programmatic Configuration**
 

src/main/java/io/r2dbc/postgresql/PostgresqlConnectionConfiguration.java

@@ -345,7 +345,7 @@ public static final class Builder {
 
         private boolean tcpKeepAlive = false;
 
-        private boolean tcpNoDelay = false;
+        private boolean tcpNoDelay = true;
 
         @Nullable
         private LoopResources loopResources = null;

src/test/java/io/r2dbc/postgresql/PostgresqlConnectionConfigurationUnitTests.java

@@ -87,7 +87,7 @@ void configuration() {
             .username("test-username")
             .sslMode(SSLMode.ALLOW)
             .tcpKeepAlive(true)
-            .tcpNoDelay(true)
+            .tcpNoDelay(false)
             .loopResources(loopResources)
             .build();
 
@@ -102,7 +102,7 @@ void configuration() {
             .hasFieldOrPropertyWithValue("username", "test-username")
             .hasFieldOrProperty("sslConfig")
             .hasFieldOrPropertyWithValue("tcpKeepAlive", true)
-            .hasFieldOrPropertyWithValue("tcpNoDelay", true)
+            .hasFieldOrPropertyWithValue("tcpNoDelay", false)
             .hasFieldOrPropertyWithValue("loopResources", loopResources);
 
         assertThat(configuration.getOptions())
@@ -131,7 +131,7 @@ void configurationDefaults() {
             .hasFieldOrPropertyWithValue("username", "test-username")
             .hasFieldOrProperty("sslConfig")
             .hasFieldOrPropertyWithValue("tcpKeepAlive", false)
-            .hasFieldOrPropertyWithValue("tcpNoDelay", false)
+            .hasFieldOrPropertyWithValue("tcpNoDelay", true)
             .hasFieldOrPropertyWithValue("loopResources", null);
 
         assertThat(configuration.getOptions())