On Mon, 3 Apr 2023 14:29:29 GMT, Daniel Jeliński <[email protected]> wrote:
>> test/jdk/com/sun/net/httpserver/ExchangeAttributeTest.java line 29: >> >>> 27: * @summary Tests for HttpExchange set/getAttribute >>> 28: * @library /test/lib >>> 29: * @run junit/othervm ExchangeAttributeTest >> >> Hello Daniel, I didn't see anything specific in the test which would need >> `othervm`. Is it the use of `ConsoleHandler` against the >> `com.sun.net.httpserver` logger which prompted the use of `othervm`? > > Actually, I wanted to make sure the HttpClient will be closed as soon as > possible. And yes, the logger would also leak to other tests. Sounds fine to me then. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13264#discussion_r1156058981
