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

   ## What is the purpose of the change
   
   Backport of #29373 to release-2.3.
   
   ## Verifying this change
   
   - `./mvnw -pl flink-table/flink-table-runtime -am -Pskip-webui-build 
-Dtest='NonTimeRowsUnboundedPrecedingFunctionTest,NonTimeRangeUnboundedPrecedingFunctionTest'
 -Dsurefire.failIfNoSpecifiedTests=false package` (JDK 17): BUILD SUCCESS, 
`NonTimeRangeUnboundedPrecedingFunctionTest` `Tests run: 26, Failures: 0, 
Errors: 0, Skipped: 1`, `NonTimeRowsUnboundedPrecedingFunctionTest` `Tests run: 
28, Failures: 0, Errors: 0, Skipped: 2`
   - `./mvnw -q -pl flink-table/flink-table-runtime spotless:check 
checkstyle:check` (JDK 17): passes
   - Same test command with the change to 
`NonTimeRowsUnboundedPrecedingFunction` reverted (JDK 17): BUILD FAILURE, 
`testInsertWithDuplicateSortKeyAndLastValueAgg` fails with 
`ArrayIndexOutOfBoundsException` on both state backends and 
`testRetractWithEarlyOutDoesNotRestartFollowingSortKeys` fails on heap
   
   ## Difference from the original
   
   | Original commit | Conflict | Byte-identical | Changes |
   | --- | --- | --- | --- |
   | 7a76f129672 | No | Yes | None |
   
   ## Notes
   
   - The 3 skipped results are the RocksDB runs of the early-out tests, stopped 
by the `assumeEarlyOutSupported()` assumption that #29373 itself adds. Their 
heap runs pass.
   
   cc @bvarghese1 @MartijnVisser
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes (please specify the tool below)
   
   Generated-by: Claude Code (Claude Opus 5.5)
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   


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