On Tue, 10 Mar 2026 10:30:43 GMT, Volkan Yazici <[email protected]> wrote:
>> Jaikiran Pai has updated the pull request incrementally with one additional >> commit since the last revision: >> >> add comment explaining othervm usage > > test/jdk/com/sun/net/httpserver/TaskRejectedTest.java line 48: > >> 46: * @bug 8169358 >> 47: * @summary HttpServer does not close client connection when >> RejectedExecutionException occurs. >> 48: * @run main/othervm TaskRejectedTest > > Can we also document the choice for `othervm`, since it is not self-evident? > For instance, > > > @comment Since loggers are modified, `othervm` is added to disallow the reuse > of this JVM > > > Note that this remark applies to multiple places in this PR. Done, I've updated the PR to add a `@comment` to relevant test classes. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/30158#discussion_r2912524770
