abstractdog commented on PR #498:
URL: https://github.com/apache/tez/pull/498#issuecomment-4471931369

   > * 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.
   ```


-- 
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]

Reply via email to