Re: [8u-dev] Request for review and approval: 8146086: Publishing two webservices on same port fails with "java.net.BindException: Address already in use"

2016-12-20 Thread Seán Coffey

Looks fine Aleksej.

Approved for jdk8u-dev.

regards,
Sean.

On 19/12/2016 15:53, Aleks Efimov wrote:

Hi,

Can I, please, ask JDK8 reviewer to look through the backported changes?

Thanks,
Aleksej

On 15/12/16 17:06, Aleks Efimov wrote:

Hi,
Can I, please, have an approval to backport JDK-8146086 to 8u-dev.
The source fix was slightly modified to remove stream/lamdba usages, 
therefore the fix is a subject to review.

Webrev with 8u-dev changes:
http://cr.openjdk.java.net/~aefimov/8146086/8/00/

The fix was tested with JPRT on all platforms - no failures observed.

Best Regards,
Aleksej

JBS:
https://bugs.openjdk.java.net/browse/JDK-8146086

JDK9 review thread:
http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-January/037984.html 



JDK9 changesets:
http://hg.openjdk.java.net/jdk9/dev/jaxws/rev/daffd583eb35
http://hg.openjdk.java.net/jdk9/dev/jdk/rev/099e432fe59c







Re: [8u-dev] Request for review and approval: 8146086: Publishing two webservices on same port fails with "java.net.BindException: Address already in use"

2016-12-19 Thread Aleks Efimov

Hi,

Can I, please, ask JDK8 reviewer to look through the backported changes?

Thanks,
Aleksej

On 15/12/16 17:06, Aleks Efimov wrote:

Hi,
Can I, please, have an approval to backport JDK-8146086 to 8u-dev.
The source fix was slightly modified to remove stream/lamdba usages, 
therefore the fix is a subject to review.

Webrev with 8u-dev changes:
http://cr.openjdk.java.net/~aefimov/8146086/8/00/

The fix was tested with JPRT on all platforms - no failures observed.

Best Regards,
Aleksej

JBS:
https://bugs.openjdk.java.net/browse/JDK-8146086

JDK9 review thread:
http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-January/037984.html 



JDK9 changesets:
http://hg.openjdk.java.net/jdk9/dev/jaxws/rev/daffd583eb35
http://hg.openjdk.java.net/jdk9/dev/jdk/rev/099e432fe59c