> Please review a jstat test-only fix.
> 
> 1. Ensure that jstat tests that run more than one sub-test per test file 
> check the result of each sub-test before continuing.
> 2. Allow '-' as a valid jstat -gcutil heap space percent occupancy field 
> value, see lineCounts[1-4].awk. Occupancy is computed as ((capacity - used) / 
> capacity), see 
> src/jdk.jcmd/share/classes/sun/tools/jstat/resources/jstat_options, search 
> for "gcutil". G1, ZGC, and Shenandoah may return zero capacity for any or all 
> of survivor spaces, eden, old gen, metaspace, and compressed class space, 
> leading to an occupancy result represented by '-'.
> 
> Modified tests pass.

Paul Hohensee 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 two additional 
commits since the last revision:

 - Merge branch 'master' into 8279005
 - 8279005: sun/tools/jstat tests do not check for test case exit codes after 
JDK-8245129

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/25951/files
  - new: https://git.openjdk.org/jdk/pull/25951/files/201f1f5e..1b9b31b6

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=25951&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25951&range=00-01

  Stats: 2863 lines in 190 files changed: 1035 ins; 520 del; 1308 mod
  Patch: https://git.openjdk.org/jdk/pull/25951.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/25951/head:pull/25951

PR: https://git.openjdk.org/jdk/pull/25951

Reply via email to