Aggarwal-Raghav commented on PR #498: URL: https://github.com/apache/tez/pull/498#issuecomment-4471938840
> > * Containerized Yetus Execution: Configured Apache Yetus to run entirely within an isolated Docker container using `build-tools/docker/Dockerfile` ensuring a reproducible build environment. > > * Removed Legacy Scripts: Deleted old custom shell scripts (build-tools/smart-apply-patch.sh and build-tools/test-patch.sh), delegating test orchestration entirely to upstream Apache Yetus. > > * Introduced Custom Yetus Personality: Created `dev-support/tez-personality.sh` to centralize and strictly control Tez-specific test execution logic, moving intelligence out of the Jenkinsfile. > > * Enforced Global Builds & Unit Tests: Configured the Yetus personality to unconditionally execute the compile and unit test phases on the root module for every PR. This guarantees that cross-module regressions are caught, even for infrastructure-only changes. > > * Smart Static Analysis: Optimized `spotbugs, checkstyle, javac, and javadoc` to run "smartly" (only on changed modules) to improve build times, while forcing full-project checks if critical files (like .proto or pom.xml) are modified. > > * Improved Artifact Archiving: Added post-build steps to automatically zip and archive all surefire-reports, keeping the Jenkins console clean while making granular test debugging much easier. > > > > **Flags REMOVED:** > > > > * --skip-dirs=`dev-support`: Removed to allow Yetus to properly analyze and lint changes made to the CI infrastructure scripts themselves. > > * --shelldocs=...`/dev-support/bin/shelldocs`: Removed because the path doesn't exists; Use shellcheck bundled in the Docker image. > > * --proclimit=5500: Removed from CLI arguments. (Migrated to `PROC_LIMIT` inside `tez-personality.sh`). > > * --dockermemlimit=20g: Removed from CLI arguments. (Migrated to `DOCKER_MEMORY` inside `tez-personality.sh`). > > * --debug: Removed to reduce log spam in standard CI runs. > > > > **Flags ADDED:** > > > > * --personality=`.../tez-personality.sh`: Added to load the newly created Tez-specific build and filtering logic. > > just for clarity's sake, this is not a new thing, right? > > ``` > Containerized Yetus Execution: Configured Apache Yetus to run entirely within an isolated Docker container using build-tools/docker/Dockerfile ensuring a reproducible build environment. > ``` yeah yeah. --docker flag was present earlier as well -- 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]
