RFR: 8286066: assert(k != __null) failed: klass not loaded caused by FillerObject_klass [v8]

[Ioi Lam]

On Thu, 5 May 2022 02:34:39 GMT, David Holmes <dholmes at openjdk.org> wrote:

>>> Use ` TestCommon.getCurrentArchiveName()` to get a unique name to use.
>> 
>> Hi @dholmes-ora , I didn't get the point why we need to get a unique name.
>> What's the problem if we always use the same name?
>> Thanks.
>
> We had issues in the past with tests running concurrently IIRC . I'm not sure if this is still an issue if the test working directory is already unique. @iklam should remember the details. We put in a lot of support code to make sure things always worked smoothly.

We had problems on Windows in the past. If two tests generate an archive with the same name, after the first test finishes, jtreg deletes the first version of this file. However, Windows virus scanners will keep the file alive for a little longer. When the second tests tries to create the file again, it gets a file permission error. This would lead to infrequent random failures in the CDS tests.

That's the reason we use `TestCommon.getCurrentArchiveName()` in the CDS tests.

-------------

PR: https://git.openjdk.java.net/jdk/pull/8519