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

Daniel Fuchs daniel.fuchs at oracle.com
Fri May 24 14:02:52 UTC 2019


Hi Vyom,

On 24/05/2019 14:36, Vyom Tiwari wrote:
> Hi Daniel,
> Overall changes looks good to me, please update the copy write date that 
> you missed in couple of files.

Which files did I miss?

> In "TestHttpServer.java"  we are printing 
> error message on System.err and throwing the RuntimeException both as 
> below, do you think we need both ?
> 
>   System.err.println ("Server could not start: " + e);
> + throw new RuntimeException("Server could not start: " + e, e);

Yes - if we don't throw the test will fail in timeout instead.
I prefer to fail fast. The exception might be swallowed if the
server is started in an executor thread though - so let's keep
both.

best regards.

-- daniel

> 
> Thanks,
> Vyom
> 
> On Fri, May 24, 2019 at 6:33 PM Chris Hegarty <chris.hegarty at oracle.com 
> <mailto:chris.hegarty at oracle.com>> wrote:
> 
> 
>     On 22/05/2019 18:35, Daniel Fuchs wrote:
>      > Hi,
>      >
>      > Please find below a patch for the next batch of tests
>      > that have been observed failing intermittently.
>      >
>      > JBS: https://bugs.openjdk.java.net/browse/JDK-8224603
>      >
>      > webrev: http://cr.openjdk.java.net/~dfuchs/webrev_8224603/webrev.00/
> 
>     Looks ok to me Daniel.
> 
>     -Chris.
> 
> 
> 
> -- 
> Thanks,
> Vyom



More information about the net-dev mailing list