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