1996fanrui opened a new pull request, #29878:
URL: https://github.com/apache/flink/pull/29878

   ## What is the purpose of the change
   
   Backport of #28573 to release-1.20.
   
   `YarnTestBase.WHITELISTED_STRINGS` on release-1.20 still only accepts the 
bare `java.net.ConnectException` cause in the gated re-association WARN, so the 
Netty-wrapped `AbstractChannel$AnnotatedConnectException` form trips 
`checkForProhibitedLogContents` (e.g. 
`YARNSessionFIFOSecuredITCase.testDetachedMode`), as reported for this branch 
in the Jira.
   
   ## Verifying this change
   
   - `./mvnw -pl flink-yarn-tests -am -Pskip-webui-build 
-Dtest='YarnTestBaseTest' -Dsurefire.failIfNoSpecifiedTests=false package` (JDK 
11), run 5 times: `BUILD SUCCESS`, each run `Tests run: 2, Failures: 0, Errors: 
0, Skipped: 0 -- in org.apache.flink.yarn.YarnTestBaseTest`
   - Same command with the `YarnTestBase.java` change reverted (new probes 
kept): `Tests run: 2, Failures: 1`, `ensureWhitelistEntryMatches` fails with 
"The following string didn't match any whitelisted patterns" on the 
Netty-wrapped line; change restored afterwards.
   - `./mvnw -q -pl flink-yarn-tests spotless:check checkstyle:check` (JDK 11): 
passes
   
   ## Difference from the original
   
   | Original commit | Conflict | Byte-identical | Changes |
   | --- | --- | --- | --- |
   | e91a0ce09e5 | No | Yes | None |
   
   ## Notes
   
   - The YARN ITCases themselves (`YARNSessionFIFOSecuredITCase` etc.) were not 
run locally: they need `verify` and the failure is intermittent, so a local run 
cannot show the fix. The change is test-only and covered by `YarnTestBaseTest`.
   
   ##### Was generative AI tooling used to co-author this PR?
   
   <!--
   If generative AI tooling has been used in the process of authoring this PR, 
please
   change the checkbox below to `[X]` followed by the name of the tool, and 
uncomment the
   "Generated-by" line. See the ASF Generative Tooling Guidance for details:
   https://www.apache.org/legal/generative-tooling.html
   
   You are responsible for the quality and correctness of every change in this 
PR
   regardless of the tooling used. Low-effort AI-generated PRs will be closed. 
See
   AGENTS.md for the full guidance.
   -->
   
   - [X] Yes (please specify the tool below)
   
   Generated-by: Claude Code (Opus 5.5)
   


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