[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 17:41:59 UTC 2019
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
>
More information about the net-dev
mailing list