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