On Fri, 16 Oct 2026 10:20:45 GMT, Jaikiran Pai <[email protected]> wrote:
> Can I please get a review of this change which adds some debug logs to the > `test/jdk/java/net/SocketOption/OptionsTest.java` test? This should help > debug which socket option is causing failures in the test runs in our CI. > > I considered changing this to a junit test, but it will require much more > work and isn't straightforward. So for now, this trivial change should help > easily debug any failures. With this change, the logs will contain message > like: > > > java.net.Socket - Test SocketOption: SO_KEEPALIVE with value: true > java.net.Socket - Test SocketOption: SO_SNDBUF with value: 1000 > java.net.Socket - Test SocketOption: SO_RCVBUF with value: 800 > java.net.Socket - Test SocketOption: SO_REUSEADDR with value: false > java.net.Socket - Test SocketOption: SO_REUSEPORT with value: false > ... > > > > --------- > - [x] I confirm that I make this contribution in accordance with the [OpenJDK > Interim AI Policy](https://openjdk.org/legal/ai). This pull request has now been integrated. Changeset: 3dc8346e Author: Jaikiran Pai <[email protected]> URL: https://git.openjdk.org/jdk/commit/3dc8346e79a251138ad587118a443492acf560c8 Stats: 7 lines in 1 file changed: 7 ins; 0 del; 0 mod 8394751: Improve test/jdk/java/net/SocketOption/OptionsTest.java for debuggability Reviewed-by: vyazici ------------- PR: https://git.openjdk.org/jdk/pull/33408
