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

   ## What is the purpose of the change
   
   Backport of #29391 to release-2.3.
   
   ## Verifying this change
   
   - `./mvnw -pl flink-runtime -am -Pskip-webui-build 
-Dtest=CreditBasedPartitionRequestClientHandlerTest 
-Dsurefire.failIfNoSpecifiedTests=false package` (JDK 17): `Tests run: 49, 
Failures: 0, Errors: 0, Skipped: 0`, `BUILD SUCCESS`
   - Same command with the change to `CreditBasedPartitionRequestClientHandler` 
reverted (JDK 17): `Tests run: 49, Failures: 2, Errors: 0, Skipped: 0`. The two 
failures are `testChannelInactiveErrorContainsLocalAndRemoteAddress` and 
`testConnectionResetErrorContainsLocalAndRemoteAddress`. After restoring the 
change the run passes again.
   - `./mvnw -q -pl flink-runtime spotless:check checkstyle:check` (JDK 17): 
passed
   
   ## Difference from the original
   
   | Original commit | Conflict | Byte-identical | Changes |
   | --- | --- | --- | --- |
   | 4d87a7d2489 | Yes | No | Only context lines differ. The test's import 
block conflicted because release-2.3 has `java.util.ArrayDeque` next to the new 
`java.net.SocketAddress` import; both are kept, in sorted order. No added or 
removed line differs from the original. |
   
   ## Notes
   
   - FLINK-40903 is an Improvement with fix version 2.3.1, which is why it is 
backported to this branch.
   
   cc @skywalker0618 @HuangZhenQiu
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes (please specify the tool below)
   
   Generated-by: Claude Code 2.1.316 (Claude 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