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

Chris Yin xu.y.yin at oracle.com
Mon Jul 16 00:50:39 UTC 2018


Thank you, Vyom

Regards,
Chris

> On 13 Jul 2018, at 5:15 PM, vyom tewari <vyom.tewari at oracle.com> wrote:
> 
> Hi Chris,
> 
> latest webrev looks good to me, thanks for explanation about copyright date.
> 
> Thanks,
> 
> Vyom
> 
> On Friday 13 July 2018 11:44 AM, Chris Yin wrote:
>> Hi, Vyom
>> 
>> Thank you for the review and comments, update webrev as below and comment inline
>> 
>> webrev: http://cr.openjdk.java.net/~xyin/8198882/webrev.02/ <http://cr.openjdk.java.net/%7Exyin/8198882/webrev.02/>
>> 
>> 
>>> On 13 Jul 2018, at 1:46 PM, vyom tewari <vyom.tewari at oracle.com <mailto:vyom.tewari at oracle.com>> wrote:
>>> 
>>> 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();
>> Fixed, thanks
>> 
>>> 
>>> 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 for checking this. Since this task is part of umbrella enhancement JDK-8191011 <https://bugs.openjdk.java.net/browse/JDK-8191011> JNDI SQE tests co-location, for those added tests which are migrated from SQE tests, the copyright date will follow the guidance SQE Test copyright year + migration copyright year if the 2 year are not same, for dump files (like *.dns) are new added under our new framework so just use current copyright year, hope that explains :), thanks
>> 
>> Regards,
>> Chris
>> 
>>> 
>>> 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 <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