> 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