[teststabilization] RFR: 8223798 : Replace wildcard address with loopback or local host in tests - part 7

Aleks Efimov aleksej.efimov at oracle.com
Wed May 15 18:30:58 UTC 2019


Vyom, Daniel,

Thanks for your reviews!

Best Regards,
Aleksei

On 15/05/2019 18:51, Vyom Tiwari wrote:
> Hi Aleks,
> latest changes looks good to me .
> Thanks,
> Vyom
>
> On Wed, May 15, 2019 at 11:12 PM Aleks Efimov 
> <aleksej.efimov at oracle.com <mailto:aleksej.efimov at oracle.com>> wrote:
>
>     Hi Daniel,
>
>     Thanks for the review. I've modified
>     Socket_getInputStream_[read|write]
>     to follow your suggestion:
>     -            ServerSocket ss = new ServerSocket(0);
>                   InetAddress lh = InetAddress.getLocalHost();
>     +            ServerSocket ss = new ServerSocket(0, 0, lh);
>
>     Also I've broke the long lines in few places. Will push the
>     changes shortly.
>
>     With Best Regards,
>     Aleksei
>
>     On 15/05/2019 17:59, Daniel Fuchs wrote:
>     > Hi Aleksei,
>     >
>     > On 15/05/2019 17:07, Aleks Efimov wrote:
>     >> Hi,
>     >>
>     >> Another part of test fixes to address intermittent networking test
>     >> failures can be viewed here:
>     >> http://cr.openjdk.java.net/~aefimov/8223798/00/
>     >
>     > Socket_getInputStream_read.java:
>     > Socket_getOutputStream_write.java:
>     >
>     > I think you could simply move the initialization of
>     > lh above the line that creates the server socket
>     > and bind the server socket to lh instead. That
>     > would keep the changes minimal.
>     >
>     > The rest looks good - but there are a few long lines.
>     > If you can break them before pushing that would be good.
>     >
>     > best regards,
>     >
>     > -- daniel
>     >
>     >>
>     >> Could I please ask for the help to review it?
>     >>
>     >> JBS:
>     >> https://bugs.openjdk.java.net/browse/JDK-8223798
>     >>
>     >>
>     >> With Best Regards,
>     >> Aleksei
>     >
>
>
>
> -- 
> Thanks,
> Vyom

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/net-dev/attachments/20190515/d37d05d7/attachment.html>


More information about the net-dev mailing list