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).

-------------

Commit messages:
 - 8394751: Improve test/jdk/java/net/SocketOption/OptionsTest.java for 
debuggability

Changes: https://git.openjdk.org/jdk/pull/33408/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=33408&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8394751
  Stats: 7 lines in 1 file changed: 7 ins; 0 del; 0 mod
  Patch: https://git.openjdk.org/jdk/pull/33408.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/33408/head:pull/33408

PR: https://git.openjdk.org/jdk/pull/33408

Reply via email to