Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

JavaTestRunner does not have enough heap to always process results #4534

Open
andrew-m-leonard opened this issue Apr 26, 2023 · 2 comments
Open

Comments

@andrew-m-leonard
Copy link
Contributor

jdk17 Win32 jck-compiler-lang-ANNOT_0 runs out of heap space when JavaTestRunner processes the result summary, due to the size of the group:

22:44:59  Test results: passed: 6,617
22:44:59  
22:44:59  Results written to C:\Users\jenkins\workspace\Test_openjdk17_hs_extended.jck_x86-32_windows\aqa-tests\TKG\output_16824520094147\jck-compiler-lang-ANNOT_0\workdir
22:44:59  Report written to C:\Users\jenkins\workspace\Test_openjdk17_hs_extended.jck_x86-32_windows\aqa-tests\TKG\output_16824520094147\jck-compiler-lang-ANNOT_0\report
22:45:06  java.lang.OutOfMemoryError: Java heap space
22:45:06  	at java.xml/com.sun.org.apache.xerces.internal.dom.DeferredDocumentImpl.getNodeObject(DeferredDocumentImpl.java:942)
22:45:06  	at java.xml/com.sun.org.apache.xerces.internal.dom.DeferredElementImpl.synchronizeData(DeferredElementImpl.java:122)
22:45:06  	at java.xml/com.sun.org.apache.xerces.internal.dom.ElementImpl.getTagName(ElementImpl.java:332)
22:45:06  	at java.xml/com.sun.org.apache.xerces.internal.dom.DeepNodeListImpl.nextMatchingElementAfter(DeepNodeListImpl.java:198)
22:45:06  	at java.xml/com.sun.org.apache.xerces.internal.dom.DeepNodeListImpl.item(DeepNodeListImpl.java:146)
22:45:06  	at java.xml/com.sun.org.apache.xerces.internal.dom.DeepNodeListImpl.getLength(DeepNodeListImpl.java:116)
22:45:06  	at JavaTestRunner.generateSummary(JavaTestRunner.java:1099)
22:45:06  	at JavaTestRunner.main(JavaTestRunner.java:184)
@smlambert
Copy link
Contributor

Tangentially related, we are looking to move away from JavaTestRunner and towards calling javatest executable more directly (see related: #4512).

@adamfarley
Copy link
Contributor

adamfarley commented Apr 26, 2023

Here's a run without the max heap increase for the test itself, helping to isolate the heap increase for JavaTestRunner as the workaround that solved this.
Update: Passed. Yup, it's just the JavaTestRunner heap. Good to be sure. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

No branches or pull requests

3 participants