Re: JEP 110 moving implementation to incubator module
On 8 Dec 2016, at 18:12, Michael McMahon wrote: > > Hi, > > Bug id 8170648 was filed to do the move and also to bring other work > done in the sandbox back into jdk9/dev. > > The following are the webrevs for the top level repo and for jdk > for this change. > > http://cr.openjdk.java.net/~michaelm/8170648/jdk/webrev/ > > and > > http://cr.openjdk.java.net/~michaelm/8170648/top/webrev/ > > We intend to push these changesets tomorrow if possible. Thank you Michael. I agree with the approach been taken here, and the changes look in line with what I expected to see. -Chris.
JEP 110 moving implementation to incubator module
Hi, Bug id 8170648 was filed to do the move and also to bring other work done in the sandbox back into jdk9/dev. The following are the webrevs for the top level repo and for jdk for this change. http://cr.openjdk.java.net/~michaelm/8170648/jdk/webrev/ and http://cr.openjdk.java.net/~michaelm/8170648/top/webrev/ We intend to push these changesets tomorrow if possible. Thanks, Michael
Re: RFR 8169115, java/net/InetAddress/ptr/lookup.sh failed intermittently
Langer and Dmitry,
thanks for the review. Pushed with typo fixed
-Felix
On 2016/12/8 20:21, Langer, Christoph wrote:
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 ; core-libs-
d...@openjdk.java.net; net-dev@openjdk.java.net; CHRIS.HEGARTY
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
RE: RFR 8169115, java/net/InetAddress/ptr/lookup.sh failed intermittently
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 ; core-libs-
> d...@openjdk.java.net; net-dev@openjdk.java.net; CHRIS.HEGARTY
>
> 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
> >>
> >
Re: RFR 8169115, java/net/InetAddress/ptr/lookup.sh failed intermittently
Felix, Changes looks good to me. Thank you for doing it. -Dmitry On 2016-12-08 13:35, Felix Yang wrote: > 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 >>> >> > -- Dmitry Samersoff Oracle Java development team, Saint Petersburg, Russia * I would love to change the world, but they won't give me the sources.
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
