Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
perf: enable tcpNoDelay by default (#2495)
In [#2341](#2341) the option to configure tcpNoDelay was added, with a default value of false. The previous value of this setting was true, as set in PGStream.changeSocket(Socket) with the following code: ``` // Submitted by Jason Venner <jason@idiom.com>. Disable Nagle // as we are selective about flushing output only when we // really need to. connection.setTcpNoDelay(true); ``` `changeSocket()` is called from the `PGStream` constructor, which is called in `ConnectionFactoryImpl`, and then later we overwrite the value with the one from the properties. We had a performance issue when upgrading from 42.2.24 to 42.3.3, and we suspected the issue in 42.3.2 and confirmed that by passing the property `tcpNoDelay=true` to the driver, the performance issue was fixed.
- Loading branch information