abstractdog opened a new pull request, #6953: URL: https://github.com/apache/hive/pull/6953
### What changes were proposed in this pull request? - A minimal Alpine-based precommit image in `dev-support/docker/precommit`: a Temurin 21 JDK, Maven, a pruned Hadoop client (only what `schematool` needs) and the shell tools the precommit stages and tests use. It is 194MB compressed. - A GitHub Actions workflow (`precommit-image.yml`). It builds and smoke tests the image on PRs that change it. On master it publishes the image for amd64 and arm64 as `apache/hive:precommit`. - The Jenkinsfile now runs in `apache/hive:precommit` instead of `ayushtkn/hive-dev-box:executor`. The dev-box-specific lines (`sw java`, `/etc/profile.d/*.sh`) are dropped, and an `IMAGE` build parameter lets a run use a candidate image. - The metastore init stages moved into `dev-support/precommit-init-metastore.sh`, so they can also be run locally. - A README with step-by-step instructions for changing, testing and publishing the image. ### Why are the changes needed? The precommit image used to be built from a separate repository (hive-dev-box) and pushed to a personal Docker Hub account. Keeping the Dockerfile in the Hive repository and publishing it from CI makes it reviewable and reproducible. It also removes the dependency on a personal account. See HIVE-29591. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? Locally, in the image with a `docker:dind` sidecar, following the README: - `smoke-test.sh` passes. - `mvn install -DskipTests -Pitests,dist` succeeds. - `init@derby` and `init@postgres` succeed. - qtests pass: `newline.q` (python `TRANSFORM`) on `TestMiniLlapCliDriver` and `jdbc_project_pushdown.q` (postgres in docker) on `TestMiniLlapLocalCliDriver`. -- 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]
