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

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


On Mon, 7 Dec 2020 12:29:21 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:

> 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

This pull request has now been integrated.

Changeset: e08b9ed0
Author:    Thomas Schatzl <tschatzl at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/e08b9ed0
Stats:     59 lines in 1 file changed: 0 ins; 59 del; 0 mod

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

Reviewed-by: jiefu, kbarrett

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

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



More information about the hotspot-gc-dev mailing list