httpserver does not close connections when RejectedExecutionException occurs

Daniel Fuchs daniel.fuchs at oracle.com
Wed Mar 7 15:56:13 UTC 2018


Hi Yuji,

On 07/03/2018 01:07, KUBOTA Yuji wrote:
> Nice catch! I removed this unused variable.
> http://cr.openjdk.java.net/~ykubota/8169358/webrev.04/
> 
> Best regards
> Yuji
> 

This looks good to me as well. I have imported your patch
and sent it to our test system and things are looking fine.

Could you prepare a final changeset with a properly formatted
comment [1], use jcheck [2] to verify that your changeset conforms
to the OpenJDK rules (no trailing whitespaces, no tabs etc...),
and upload it somewhere to http://cr.openjdk.java.net/~ykubota/8169358 ?

If there aren't any other review feedback in the mean time, I'll
be happy to sponsor your changes when that's ready.

best regards,

-- daniel
[1] http://openjdk.java.net/guide/producingChangeset.html
[2] http://openjdk.java.net/projects/code-tools/jcheck/



More information about the net-dev mailing list