RFR 8198882: Add 10 JNDI tests to com/sun/jndi/dns/AttributeTests/

vyom tewari vyom.tewari at oracle.com
Fri Jul 13 05:46:32 UTC 2018


Hi Chris,

Thanks for doing this overall looks good to me, few minor comments.

1->  DNSTestUtils.java, please start the server first and then set the 
"TEST_DNS_SERVER_THREAD". This will not make much difference but we will 
avoid setting "TEST_DNS_SERVER_THREAD" env variable if server fails to 
start.

129             env.put(TEST_DNS_SERVER_THREAD, inst);
  130             inst.start();


2->  I noticed that  copyright date (Copyright (c) 2000, 2018,) but 
webrev tells all the tests are new, please fix copyright date as well.

Thanks,
Vyom

On Thursday 12 July 2018 02:08 PM, Chris Yin wrote:
> Please have a review to new webrev as below, some code refactoring on 
> lib parts and enhanced DNSServer to handle retry request which will 
> make the tests more stable, thanks
>
> http://cr.openjdk.java.net/~xyin/8198882/webrev.01/ 
> <http://cr.openjdk.java.net/%7Exyin/8198882/webrev.01/>
>
> Regards,
> Chris
>
>> On 22 Mar 2018, at 11:16 AM, Chris Yin <xu.y.yin at oracle.com 
>> <mailto:xu.y.yin at oracle.com>> wrote:
>>
>> Please review the changes to add 10 JNDI tests to 
>> com/sun/jndi/dns/AttributeTests/, thanks
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8198882
>> webrev: http://cr.openjdk.java.net/~xyin/8198882/webrev.00/ 
>> <http://cr.openjdk.java.net/%7Exyin/8198882/webrev.00/>
>>
>> Regards,
>> Chris
>



More information about the core-libs-dev mailing list