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

Vyom Tiwari vyommani at gmail.com
Fri May 24 14:37:46 UTC 2019


Hi Daniel,

SocksProxyVersion.java is one of them my local repo it is showing
(Copyright (c) 2011, 2015, Oracle and/or its). I was talking do we need
both(printing message on System.err & throwing exception), as you already
explain throwing exception is required but do we need  same error
message ("Server
could not start:") on System.err as well ?
Thanks,
Vyom

On Fri, May 24, 2019 at 7:32 PM Daniel Fuchs <daniel.fuchs at oracle.com>
wrote:

> 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
>
>

-- 
Thanks,
Vyom
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.java.net/pipermail/net-dev/attachments/20190524/d2252d11/attachment.html>


More information about the net-dev mailing list