RFR: 8203925: tools/javac/importscope/T8193717.java ran out of java heap

Vicente Romero vromero at openjdk.java.net
Thu Jan 14 21:29:06 UTC 2021


On Fri, 8 Jan 2021 08:12:45 GMT, Guoxiong Li <github.com+13688759+lgxbslgx at openjdk.org> wrote:

> Hi all,
> 
> Because [JDK-8225054-PATCH](https://github.com/openjdk/jdk/commit/827e5e32264666639d36990edd5e7d0b7e7c78a9) modified `T8193717.java`, changing `private static final int CLASSES = 500000;` to `private static final int CLASSES = 50000;`(500K -> 50K). The error, `out of java heap`, no longer occurs now. And the pre-submit tests of this patch passed. I think `tools/javac/importscope/T8193717.java` could be removed from `ProblemList.txt`.
> 
> Thank you for taking the time to review.
> 
> Best Regards.

test/langtools/ProblemList.txt line 52:

> 50: tools/javac/warnings/suppress/TypeAnnotations.java                              8057683    generic-all    improve ordering of errors with type annotations
> 51: tools/javac/modules/SourceInSymlinkTest.java                                    8180263    windows-all    fails when run on a subst drive
> 52: tools/javac/importscope/T8193717.java                                           8203925    generic-all    the test requires too much memory

I believe that we want to do some more research regarding why the test fails for a large number of imports

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

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


More information about the compiler-dev mailing list