Thanks, Misha!

Jiangli 

> On Sep 17, 2018, at 4:17 PM, mikhailo <mikhailo.seledt...@oracle.com> wrote:
> 
> Change looks good,
> 
> Misha
> 
> 
>> On 09/17/2018 03:15 PM, Jiangli Zhou wrote:
>> Please review the change for JEP 341 (Default CDS Archives) sub-task, 
>> JDK-8210592.
>> 
>> Currently, there are sub-sets of tiered tests running in CDS mode (defined 
>> in closed tier5 and tier6 test definitions). These tests are also executed 
>> in 'normal' mode in various tiers. GENERATE_CDS_ARCHIVE is used to create a 
>> CDS archive using the default classlist before executing those tests in 
>> CDS-mode.
>> 
>> When the default CDS archive is enabled, it is no longer necessary to 
>> execute those tests in CDS mode explicitly since all tiered testing enables 
>> the default CDS archive by default. The change in the webrev removes 
>> GENERATE_CDS_ARCHIVE. To increase test coverage, the test sets in CDS mode 
>> are converted to run in non-CDS mode (with -Xshare:off enabled explicitly) 
>> in tier5 and tier6. The conversion is done in the closed repo.
>> 
>>   webrev: http://cr.openjdk.java.net/~jiangli/8210592/webrev.00/
>> 
>>   JEP sub-task: https://bugs.openjdk.java.net/browse/JDK-8210592
>> 
>> Tested tier5 and tier6 with the default CDS archive patch enabled.
>> 
>> Thanks,
>> 
>> Jiangli
>> 
> 

Reply via email to