RFR: 8257820: Remove gc/ergonomics/TestMinHeapSize.java as it is too brittle

Thomas Schatzl tschatzl at openjdk.java.net
Mon Dec 7 12:34:17 UTC 2020


Hi all,

  can I have reviews for this change that removes the gc/ergonomics/TestMinHeapSize.java test?

We found that it is unstable, and with reasonable effort very likely can't be made to work: we have no idea how much (C heap) memory the rest of the VM will use, or will in the future, so there will likely always be a constant need to update it. Even during review there have already some adaptations to make it work, until it "worked", but then it intermittently started failing in CI anyway.

Testing: this is a trivial removal of a complete test/file.

Thanks,
  Thomas

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

Commit messages:
 - Initial commit, remove test

Changes: https://git.openjdk.java.net/jdk/pull/1664/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=1664&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8257820
  Stats: 59 lines in 1 file changed: 0 ins; 59 del; 0 mod
  Patch: https://git.openjdk.java.net/jdk/pull/1664.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/1664/head:pull/1664

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



More information about the hotspot-gc-dev mailing list