Skip to content

CI: Use Java 26#1912

Open
cstamas wants to merge 2 commits into
apache:masterfrom
cstamas:use-java-26
Open

CI: Use Java 26#1912
cstamas wants to merge 2 commits into
apache:masterfrom
cstamas:use-java-26

Conversation

@cstamas

@cstamas cstamas commented Jun 8, 2026

Copy link
Copy Markdown
Member

Up CI matrix from Java 25 to Java 26.

@cstamas

cstamas commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

@kwin this seems quite consistent: JDK transport UTs fail on Java 26?

@kwin

kwin commented Jun 9, 2026

Copy link
Copy Markdown
Member
Error:  Tests run: 85, Failures: 1, Errors: 0, Skipped: 6, Time elapsed: 4.443 s <<< FAILURE! -- in org.eclipse.aether.transport.jdk.JdkTransporterTest
Error:  org.eclipse.aether.transport.jdk.JdkTransporterTest.testRetryHandler_defaultCount_negative -- Time elapsed: 0.022 s <<< FAILURE!
org.opentest4j.AssertionFailedError: Expected error
	at org.junit.jupiter.api.AssertionUtils.fail(AssertionUtils.java:38)
	at org.junit.jupiter.api.Assertions.fail(Assertions.java:138)
	at org.eclipse.aether.transport.jdk.JdkTransporterTest.testRetryHandler_defaultCount_negative(JdkTransporterTest.java:90)
	at java.base/java.lang.reflect.Method.invoke(Method.java:565)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1612)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1612)

Seems some internal retry mechanism has changed in Java 26...

Comment thread .github/workflows/maven-verify.yml
@kwin

kwin commented Jul 2, 2026

Copy link
Copy Markdown
Member

With Java 26 the num of internal retries is 5 times the number of retries controlled via Methanol (=3 by default). Prior Java 26 the internal number of retries was 2... The retry handling got changed in openjdk/jdk@e8db14f#diff-41e3c3f66a6d78612e230f2546ac15e3f82619efe333a731b64f59a9e4991816R613.

@kwin

kwin commented Jul 2, 2026

Copy link
Copy Markdown
Member

Please rebase on top of master now @cstamas.

@cstamas cstamas self-assigned this Jul 8, 2026
@cstamas cstamas changed the title Use Java 26 CI: Use Java 26 Jul 8, 2026
@cstamas cstamas marked this pull request as ready for review July 8, 2026 11:51
@kwin

kwin commented Jul 8, 2026

Copy link
Copy Markdown
Member

Ok, still not working reliably, I would tend to disable the ITs related to retries in JDKTransporter as those are too brittle. WDYT @cstamas ?

@cstamas

cstamas commented Jul 8, 2026

Copy link
Copy Markdown
Member Author

Yup, go for it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants