The GitHub Actions job "Required Checks" on texera.git/gh-readonly-queue/main/pr-5989-40526b8e1783e7782ce6043e7ca72840b7fbe04d has failed. Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).
Head commit for run: 3ebbfeb2d18b863e21255ab2851fc6841aaf6e4f / Matthew B. <[email protected]> refactor(pyamber): share receiver-batch construction in partitioners (#5989) ### What changes were proposed in this PR? - Added `Partitioner.build_receiver_batches(channels)` to the base class, the single home for the ordered `(receiver, batch)` list built via `dict.fromkeys` (which preserves channel order where a set literal would not). - Replaced the three identical inline copies of that construction in `RoundRobinPartitioner`, `HashBasedShufflePartitioner`, and `RangeBasedShufflePartitioner` with a call to the shared helper, dropping the drifted copy-pasted comments. - No behavior change: each partitioner still builds the same ordered, deduplicated receiver list. ### Any related issues, documentation, discussions? Closes: #5988 ### How was this PR tested? - Run `cd amber && PYTHONPATH=src/main/python python -m pytest src/test/python/core/architecture/sendsemantics/test_partitioners.py -q`, expect all 35 tests to pass. - The order/dedup behavior owned by the new helper is covered by the existing `TestRoundRobinPartitioner::test_init_preserves_channel_order` and `test_init_dedupes_duplicate_channels_preserving_first_seen_order`; confirm they still pass against the refactored construction. ### Was this PR authored or co-authored using generative AI tooling? Co-authored with Claude Opus 4.8 in compliance with ASF Report URL: https://github.com/apache/texera/actions/runs/28882767876 With regards, GitHub Actions via GitBox
