[test] chore: fix flaky RemoteLogScannerITCase#3002
Open
fresh-borzoni wants to merge 1 commit intoapache:mainfrom
Open
[test] chore: fix flaky RemoteLogScannerITCase#3002fresh-borzoni wants to merge 1 commit intoapache:mainfrom
fresh-borzoni wants to merge 1 commit intoapache:mainfrom
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Fixes flakiness in RemoteLogScannerITCase.testScanFromRemoteAndProject by making assertions order-insensitive when scanning projected rows from remote log segments (addressing #2992).
Changes:
- Collects scanned records into a list and asserts using
containsExactlyInAnyOrderElementsOffor projection[a, c]. - Applies the same order-insensitive assertion approach for reordered projection
[c, a]. - Adds explicit expected-row lists (
expectedRowsAC,expectedRowsCA) to validate both projections deterministically.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Summary
closes #2992
Collect records and assert with containsExactlyInAnyOrderElementsOf, same pattern used in testScanFromRemote