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

Many "Unexpected error running resulting class file for X.java" test failures in I-build on Java 22 #2983

Closed
akurtakov opened this issue Sep 19, 2024 · 7 comments
Assignees
Labels
test Work on unit tests, no change of productive code

Comments

@srikanth-sankaran
Copy link
Contributor

@mpalat - thanks for taking a look - please reassign as suited after triage

@jukzi jukzi added the test Work on unit tests, no change of productive code label Sep 19, 2024
@stephan-herrmann
Copy link
Contributor

This is just the 21 variant of #2982 right?

@srikanth-sankaran
Copy link
Contributor

Would the practice of making sure that the BETA_JAVA_XX branch is tested with all JREs that the github CI testing uses catch these problems ahead of the merge ?

@mpalat
Copy link
Contributor

mpalat commented Sep 19, 2024

This is just the 21 variant of #2982 right?

yes.

@mpalat
Copy link
Contributor

mpalat commented Sep 19, 2024

Would the practice of making sure that the BETA_JAVA_XX branch is tested with all JREs that the github CI testing uses catch these problems ahead of the merge ?

Essentially yes, Y builds with earlier JREs - I will create an issue from Java 24 onwards to capture this

@stephan-herrmann
Copy link
Contributor

Would the practice of making sure that the BETA_JAVA_XX branch is tested with all JREs that the github CI testing uses catch these problems ahead of the merge ?

It's all here: https://ci.eclipse.org/releng/job/YPBuilds/job/ep433Y-unit-cen64-gtk3-java21/29/testReport/

I see the problem in monitoring those test results.

I personally was alarmed (admittedly late in the cycle) by seeing hundreds of failures when running BETA_JAVA23 tests on Java 23, and strictly focused on that aspect. When finally I found out that this problem was caused by a JIT bug in Java 23 and when finally I found a way to coerce that JIT not to throw NPE from null-free code, I simply had no time to check tests on other Java versions.

Ergo: monitoring test results from Y-Buids must be addressed much more actively, much earlier.

@akurtakov
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Work on unit tests, no change of productive code
Projects
None yet
Development

No branches or pull requests

5 participants