yaooqinn opened a new pull request, #11332: URL: https://github.com/apache/incubator-gluten/pull/11332
## What changes are proposed in this pull request? This PR updates the `.github/workflows/velox_backend_x86.yml` workflow to use the `build/mvn` wrapper script instead of directly calling `mvn`. **Changes:** - Updated `MVN_CMD` environment variable from `mvn -ntp` to `build/mvn -ntp` - Removed Maven installation steps from various jobs (ubuntu, centos8, centos7 test jobs) - Removed the `SETUP` environment variable and related helper script calls - Added `build/mvn` to the workflow trigger paths to ensure workflow runs when the wrapper is modified - Cleaned up Java setup steps by removing Maven installation dependencies **Benefits:** - Ensures consistent Maven version across all CI jobs - Simplifies job setup by eliminating manual Maven installation - Aligns with the project's adoption of the Maven wrapper (introduced in commit a35a3bda1) - Reduces maintenance overhead for CI configuration ## How was this patch tested? - The workflow changes follow the same pattern used in other workflows that already use `build/mvn` - The Maven wrapper script (`build/mvn`) handles Maven installation automatically - CI jobs will validate the changes when this PR is opened -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
