Hi Dmitry,
thanks for the comments. My opinions inlined...
Updated webrev: http://cr.openjdk.java.net/~xiaofeya/8169115/webrev.01/
-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)
Probably not. I'm not sure about the test design history, but
"icann.org" may require the test environment(inside Oracle) to set up
proxy to access internet, otherwise the test may be just skipped
silently. "javaweb.sfbay.sun.com" is accessible both inside and outside
Oracle network.
-Felix
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