Hi Felix,
looks good also for me.
Small typo:
85 throw new RuntimeException("Mistmatch between default"
Mistmatch -> Mismatch :)
Best regards
Christoph
> -----Original Message-----
> From: net-dev [mailto:net-dev-boun...@openjdk.java.net] On Behalf Of Felix
> Yang
> Sent: Donnerstag, 8. Dezember 2016 11:35
> To: Dmitry Samersoff <dmitry.samers...@oracle.com>; core-libs-
> d...@openjdk.java.net; net-dev@openjdk.java.net; CHRIS.HEGARTY
> <chris.hega...@oracle.com>
> Subject: Re: RFR 8169115, java/net/InetAddress/ptr/lookup.sh failed
> intermittently
>
> Hi Dmitry,
>
> I tested your suggested "icann.org" and it indeed works well. Please
> review the updated webrev:
>
> http://cr.openjdk.java.net/~xiaofeya/8169115/webrev.02/
>
> Thanks,
> Felix
> On 2016/12/6 19:16, Dmitry Samersoff wrote:
> > Felix,
> >
> > 1. I'm not sure that javaweb.sfbay.sun.com is the best domain name for
> > this test. Could we use different one (e.g. icann.org)
> >
> > 2. This test run JTREG -> Test VM -> Another VM. Could we optimize
> > process usage?
> >
> > It might be better to create a jtreg "same vm" process that
> >
> > 1. launch another process with -Djava.net.preferIPv4Stack=true
> > that do A and PRT lookup in one run.
> >
> > 2. Read results of process above, do PTR lookup with default
> > settings and compare results.
> >
> > -Dmitry
> >
> >
> > On 2016-12-06 12:06, Felix Yang wrote:
> >> Hi,
> >>
> >> please review the following patch. It generally coverts codes from
> >> shell to plain java.
> >>
> >> Bug:
> >>
> >> https://bugs.openjdk.java.net/browse/JDK-8169115
> >>
> >> Webrev:
> >>
> >> http://cr.openjdk.java.net/~xiaofeya/8169115/webrev.00/
> >>
> >> Thanks,
> >>
> >> Felix
> >>
> >
