Skip to content

Fix cache warmer: add clean to resolve maven-clean-plugin and exec-maven-plugin#1394

Merged
gopalldb merged 1 commit intomainfrom
fix/warm-cache-add-clean
Apr 8, 2026
Merged

Fix cache warmer: add clean to resolve maven-clean-plugin and exec-maven-plugin#1394
gopalldb merged 1 commit intomainfrom
fix/warm-cache-add-clean

Conversation

@gopalldb
Copy link
Copy Markdown
Collaborator

@gopalldb gopalldb commented Apr 8, 2026

Summary

maven-clean-plugin:3.2.0 and exec-maven-plugin:1.2.1 were missing from the dependency cache because the warmer ran mvn install and mvn test without the clean lifecycle phase. The CI workflows use mvn clean test, which requires these plugins.

Fix: add clean to every warmer step to match CI commands exactly.

Test plan

NO_CHANGELOG=true

This pull request was AI-assisted by Isaac.

@gopalldb gopalldb requested a review from a team as a code owner April 8, 2026 14:48
@gopalldb gopalldb force-pushed the fix/warm-cache-add-clean branch from 4545276 to 9feef8b Compare April 8, 2026 15:16
Stop guessing which plugins are needed. Instead, run the EXACT same
Maven commands from prCheck.yml, prIntegrationTests.yml, and
coverageReport.yml. This is the only reliable way to ensure every
plugin, provider, and transitive dependency is resolved and cached.

Commands mirror: spotless:check, install, Arrow tests (3 variants),
unit tests with jacoco, integration test compile, resolve-plugins.

Signed-off-by: Gopal Lal <gopal.lal@databricks.com>
Co-authored-by: Isaac
Signed-off-by: Gopal Lal <gopal.lal@databricks.com>
@gopalldb gopalldb force-pushed the fix/warm-cache-add-clean branch from 9feef8b to ec803b8 Compare April 8, 2026 15:31
@gopalldb gopalldb merged commit 2fd7949 into main Apr 8, 2026
14 of 18 checks passed
@gopalldb gopalldb deleted the fix/warm-cache-add-clean branch April 8, 2026 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants