Skip to content

chore(java-devel): Tweak Maven transport settings [skip ci] - #1584

Merged
Techassi merged 7 commits into
mainfrom
chore/tweak-maven-transport-settings
Jul 20, 2026
Merged

chore(java-devel): Tweak Maven transport settings [skip ci]#1584
Techassi merged 7 commits into
mainfrom
chore/tweak-maven-transport-settings

Conversation

@Techassi

@Techassi Techassi commented Jul 20, 2026

Copy link
Copy Markdown
Member

Note

This might get cherry-picked onto the release-26.7 branch to potentially help with failing builds in CI.

This PR tweaks various Maven transport/resolver settings to be a little more network failure resistant in the future. These network issues seem to be known by the Java/Maven community and are related to Azure's network setup.

@Techassi Techassi self-assigned this Jul 20, 2026
@Techassi Techassi moved this to Development: Waiting for Review in Stackable Engineering Jul 20, 2026
@Techassi Techassi changed the title chore(java-devel): Tweak Maven transport settimgs chore(java-devel): Tweak Maven transport settings Jul 20, 2026
Comment thread java-devel/stackable/settings.xml Outdated
Comment thread java-devel/stackable/settings.xml Outdated
Comment thread java-devel/stackable/settings.xml Outdated
Co-authored-by: Sebastian Bernauer <sebastian.bernauer@stackable.tech>
@Techassi Techassi moved this from Development: Waiting for Review to Development: In Review in Stackable Engineering Jul 20, 2026

@sbernauer sbernauer left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's see if that improves things...

@Techassi
Techassi enabled auto-merge July 20, 2026 12:20
@Techassi
Techassi disabled auto-merge July 20, 2026 12:20
@Techassi

Copy link
Copy Markdown
Member Author

Let's wait with merging this until (hopefully) we can finish building the rc1 images.

@Techassi Techassi changed the title chore(java-devel): Tweak Maven transport settings chore(java-devel): Tweak Maven transport settings [skip ci] Jul 20, 2026
@Techassi

Copy link
Copy Markdown
Member Author

All rc1 images (except a single Superset image) built.

I marked this PR with [skip ci], because #1585 will trigger builds for basically all images anyway. There is no need to waste compute on this or hammer build-repo even more.

@Techassi
Techassi added this pull request to the merge queue Jul 20, 2026
Merged via the queue into main with commit 77f25eb Jul 20, 2026
3 checks passed
@Techassi
Techassi deleted the chore/tweak-maven-transport-settings branch July 20, 2026 14:09
@Techassi Techassi moved this from Development: In Review to Development: Done in Stackable Engineering Jul 20, 2026
lfrancke added a commit that referenced this pull request Jul 21, 2026
…ed by Maven 3.9

#1584 added HTTP retry settings (retryHandler.count/interval/intervalMax) under the aether.transport.http.* keys. That prefix is the maven-resolver 2.x / Maven 4 spelling; we build with Maven 3.9.16 (maven-resolver 1.9.27), which only reads aether.connector.http.*, so those settings currently have no effect.

Add the connector.* spelling alongside the transport.* one so the retries actually apply today, mirroring how connectionMaxTtl is already dual-spelled.

CI Maven failures are connection-level (stale keep-alive connections dropped by the network; no server-side 5xx in Nexus/Apache logs). maven-resolver's retry handler retries these IOExceptions, so this makes the intended retry behaviour effective. Key names verified against ConfigurationProperties.java @ maven-resolver-1.9.27.
lfrancke added a commit that referenced this pull request Jul 21, 2026
…ed by Maven 3.9

#1584 added HTTP retry settings (retryHandler.count/interval/intervalMax) under the aether.transport.http.* keys. That prefix is the maven-resolver 2.x / Maven 4 spelling; we build with Maven 3.9.16 (maven-resolver 1.9.27), which only reads aether.connector.http.*, so those settings currently have no effect.

Add the connector.* spelling alongside the transport.* one so the retries actually apply today, mirroring how connectionMaxTtl is already dual-spelled.

CI Maven failures are connection-level (stale keep-alive connections dropped by the network; no server-side 5xx in Nexus/Apache logs). maven-resolver's retry handler retries these IOExceptions, so this makes the intended retry behaviour effective. Key names verified against ConfigurationProperties.java @ maven-resolver-1.9.27.
@lfrancke lfrancke moved this from Development: Done to Done in Stackable Engineering Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants