Aggarwal-Raghav commented on PR #498: URL: https://github.com/apache/tez/pull/498#issuecomment-4472193186
@abstractdog , regarding your above question on `test-patch` profile. Let me share my understanding: 1. By default, the standard maven-compiler-plugin hides many lint warnings and Yetus relies on doing a diff of the warnings on master vs the warnings on the PR and tells that, PR has introduced 2 more warning or fixed 2 warning for instance. Frankly, I don't see much value of that because in future if JDK25 deprecates a API then this warning diff will flag it. For Tez-1.0.0 community can take a call whether to keep track of warning or remove this profile from codebase. Below table should help in making decision 2. From my short experience with yetus, it compare things with the baseline master branch against your PR. I couldn't find a way to just run `mvninstall` yetus plugin or `compile` yetus plugin only on PR/patch and ignore master. Let me clarify a thing: 1. eliminating YETUS completley, is it acceptable? and move everything to standard Jenkinsfile? We'll lose the yetus summary output and spotbugs/checkstyle/linter which comes out of the box in yetus. Then it will be just like hive project. | Feature | Option 1 (Maximum Strictness) | Option 2 (The Sweet Spot) | Option 3 (Maximum Speed) | | :--- | :--- | :--- | :--- | | **Yetus Phases Enabled** | `mvninstall`, `compile`, `unit` | `compile`, `unit` <br>*(Disabled: `mvninstall`)* | `unit` only <br>*(Disabled: `mvninstall`, `compile`)* | | **Step 1: Baseline** | `maven install: master` <br> `master compilation: pre-patch` | `master compilation: pre-patch` | *(Skipped entirely)* | | **Step 2: Patch Build** | `maven install: patch` <br> `master compilation: patch` | `master compilation: patch` | *(Skipped entirely)* | | **Step 3: Test Execution** | `unit: patch` | `unit: patch` | `unit: patch` | | **Estimated Maven Overhead** | ~20 minutes | ~8 minutes | 0 minutes | | **Pros** | Maximum safety. Fails early on missing dependencies. | Drops JAR packaging time. Retains `test-patch` profile and early syntax failure detection and tracks compiler warning diffs. | Absolute fastest pipeline. | | **Cons** | Very slow. Creates redundant JARs that `unit` doesn't strictly need. | Very slight overhead to count warnings. | **Loses all warning tracking** (e.g., deprecations). Syntax errors fail deep in the `unit` phase instead of failing early. | -- 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]
