RFR(s): 8228580: DnsClient TCP socket timeout

Milan Mimica milan.mimica at gmail.com
Wed Sep 11 13:01:13 UTC 2019


Hi Pavel

Your changes look good to me.

On Wed, 11 Sep 2019 at 14:26, Pavel Rappo <pavel.rappo at oracle.com> wrote:
>
> I'm happy with the overall changeset. I have (once again) made some tiny changes, you can see them here:
>
>     http://cr.openjdk.java.net/~prappo/8228580/webrev.02/
>
> If you are okay with them, then we wait for a *R*eviewer. If the Reviewer(s) are okay with them, we push. For the record, I'm not really happy with how we used the DNSTestBase/TestBase infrastructure, however I'm totally fine with the retrying logic.
>
> Test results are pending.
>
> -Pavel
>
> > On 10 Sep 2019, at 16:33, Milan Mimica <milan.mimica at gmail.com> wrote:
> >
> >>> On 5 Sep 2019, at 16:02, Pavel Rappo <pavel.rappo at oracle.com> wrote:
> >>>
> >>> I think we are almost there. What do you think of the following incremental (i.e. on top of your latest webrev) change?
> >>>
> >>>   http://cr.openjdk.java.net/~prappo/8228580/webrev.01/
> >>>
> >>> I fixed a couple of trivial typos and addressed the socket relinquishing issue. Initializing a socket is not an atomic "all-or-nothing" operation now. Someone needs to take care of the socket in case things go not as planned.
> >
> > Right. Thanks. Here is the merged version:
> > http://cr.openjdk.java.net/~mmimica/8228580/webrev.02/
> > Plus, I have added TCP server init retry code from Chris. Works fine
> > without changes to TestBase.
> >
> >
> > --
> > Milan Mimica
>


-- 
Milan Mimica


More information about the core-libs-dev mailing list