JingsongLi commented on PR #10139: URL: https://github.com/apache/paimon/pull/10139#issuecomment-5805740770
The CI routing change has end-to-end value: native commit assertions should run against the matching from-source Rust bridge, while the standard Python matrix should still exercise Python fallback and non-native behavior. I checked the workflow: `Tests` sets `PYTEST_ADDOPTS=-m 'not native_plan'`, and `Rust Plan` builds paimon-rust from source and runs pytest without that exclusion. On the PR test file, collection with the exclusion selects 19 tests and deselects 38 in my local environment, confirming the marker routing. Production gate: this head currently has a failing `CI / Python / Tests / Python 3.10` check (and the aggregate CI result is failing). Please identify and resolve that failure before merge. My local checkout predates this PR base; running the fetched head test file against it produced 6 incompatibility failures, so those are not a valid regression signal for this PR. I could verify marker collection, but not the full Rust Plan runtime locally. -- 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]
