[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-serviceability-dev
Subject:    Re: RFR: 8278123: serviceability/dcmd/vm/ClassLoaderStatsTest.java failing with java.lang.AssertionE
From:       Leonid Mesnik <lmesnik () openjdk ! java ! net>
Date:       2022-04-28 21:58:46
Message-ID: fqSn47jxVZZMqCQ3scTdNWviQ_F5AVpW9GK6uhaSm4I=.77ff8bee-64e1-4a5a-91e7-3ea4da7b8c0a () github ! com
[Download RAW message or body]

> The test failed if GC happens somewhere between
> Class<?> c = Class.forName("TestClass", true, dummyloader);
> and
> OutputAnalyzer output = executor.execute("VM.classloader_stats");
> 
> The fix is to make hc static as Chris proposed. 
> 
> To verfiy fix I add System.gc() before executor.execute("VM.classloader_stats");

Leonid Mesnik has updated the pull request with a new target base due to a merge or a \
rebase. The incremental webrev excludes the unrelated changes brought in by the \
merge/rebase. The pull request contains three additional commits since the last \
revision:

 - comment fixed
 - Merge branch 'master' of https://github.com/openjdk/jdk into 8278123
 - fix

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

Changes:
  - all: https://git.openjdk.java.net/jdk/pull/8438/files
  - new: https://git.openjdk.java.net/jdk/pull/8438/files/108363b1..1e9d05a8

Webrevs:
 - full: https://webrevs.openjdk.java.net/?repo=jdk&pr=8438&range=01
 - incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=8438&range=00-01

  Stats: 22 lines in 2 files changed: 8 ins; 0 del; 14 mod
  Patch: https://git.openjdk.java.net/jdk/pull/8438.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/8438/head:pull/8438

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


[prev in list] [next in list] [prev in thread] [next in thread]