Skip to content

[ci][python] Fix lint-python native-plan CI failure - #9255

Merged
JingsongLi merged 2 commits into
apache:masterfrom
XiaoHongbo-Hope:codex/bump-pypaimon-rust-rev
Aug 17, 2026
Merged

[ci][python] Fix lint-python native-plan CI failure#9255
JingsongLi merged 2 commits into
apache:masterfrom
XiaoHongbo-Hope:codex/bump-pypaimon-rust-rev

Conversation

@XiaoHongbo-Hope

@XiaoHongbo-Hope XiaoHongbo-Hope commented Aug 16, 2026

Copy link
Copy Markdown
Contributor

Purpose

The lint-python (3.11) CI leg builds pypaimon_rust and runs the mixed native-plan tests. table_update_test.py::test_literal_predicate_update_projects_only_row_id fails deterministically on that leg (values correct, row-group order flipped):

AssertionError: Lists differ: ['NYC', 'LA', 'Updated', 'Updated', 'Updated']
                          != ['Updated', 'Updated', 'Updated', 'NYC', 'LA']

Split order across row-id groups is not a cross-planner contract: engines may process splits in any order, and the Rust native plan can return the freshly updated group first. The test asserted an unsorted read, unlike its sibling tests which sort by id before asserting.

Bumping the pinned Rust revision alone does not fix this (verified on this PR's previous CI run with be5456e): apache/paimon-rust#717 aligns file order within row-id groups, not split order across groups.

Changes

Tests

  • The two failing variants (Batch/Stream) pass locally with a native-plan-enabled pypaimon_rust.
  • flake8 passed.

@XiaoHongbo-Hope XiaoHongbo-Hope changed the title [ci] Bump pypaimon_rust rev to include Data Evolution split order fix [ci][python] Fix order-fragile predicate update test; bump pypaimon_rust rev Aug 17, 2026
@XiaoHongbo-Hope XiaoHongbo-Hope changed the title [ci][python] Fix order-fragile predicate update test; bump pypaimon_rust rev [ci][python] Fix lint-python native-plan CI failure Aug 17, 2026
@JingsongLi

Copy link
Copy Markdown
Contributor

+1

@JingsongLi
JingsongLi merged commit 8145d13 into apache:master Aug 17, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants