yaooqinn opened a new pull request, #11496:
URL: https://github.com/apache/incubator-gluten/pull/11496
## What changes were proposed in this pull request?
This PR migrates `velox_backend_enhanced.yml` and `velox_backend_arm.yml`
workflows to use the `build/mvn` wrapper instead of calling `mvn` directly.
### Changes:
- **velox_backend_arm.yml:**
- Added `'build/mvn'` to workflow trigger paths
- Changed `MVN_CMD: 'mvn -ntp'` → `'build/mvn -ntp'`
- Removed unused `SETUP` env variable
- Removed `$SETUP install_maven` from "Setup java" step (build/mvn handles
Maven auto-download)
- Renamed step: "Setup java and maven" → "Setup java"
- **velox_backend_enhanced.yml:**
- Added `'build/mvn'` to workflow trigger paths
- Changed `MVN_CMD: 'mvn -ntp'` → `'build/mvn -ntp'`
- Removed unused `SETUP` env variable
## Why are the changes needed?
The `build/mvn` wrapper provides:
1. **Automatic Maven download** - Downloads Maven version from `pom.xml` if
not present
2. **Version consistency** - Uses version defined in project's `pom.xml`
instead of hardcoded versions
3. **Reduced duplication** - Eliminates need for separate `install_maven`
helper function
This follows the pattern established in #11332 for `velox_backend_x86.yml`.
## Does this PR introduce _any_ user-facing change?
No
## How was this patch tested?
CI will validate the workflow changes.
--
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]
