On Wed, 13 Mar 2024 07:53:30 GMT, Christoph Langer <clan...@openjdk.org> wrote:
> 4f336085d1098e7fba7b58f0a73c028179a2a13d > ([JDK-8326718](https://bugs.openjdk.org/browse/JDK-8326718)) added a few > cases to test java/util/Formatter/Padding.java with huge Strings as > arguments. Since all possible argument combinations for the test are stored > in one array, nothing can be garbage collected while the test is running and > the heap requirement is blown up. > > In one of our test pipelines we run tier1 tests with VMs that default to 384M > of heap and this is not sufficient any longer. > > I'm improving this by splitting the one large @ParameterizedTest into > multiple ones. With that, I could run the test successfully in a test VM with > 96M of heap, e.g. by modifying `@run junit Padding` to `@run junit/othervm > -Xmx96m Padding` This pull request has now been integrated. Changeset: 128e60a2 Author: Christoph Langer <clan...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/128e60a29f1bd1e1fbe165ac382107070858ecc6 Stats: 650 lines in 1 file changed: 343 ins; 269 del; 38 mod 8328037: Test java/util/Formatter/Padding.java has unnecessary high heap requirement after JDK-8326718 Reviewed-by: rgiulietti ------------- PR: https://git.openjdk.org/jdk/pull/18264
