Skip to content

test: cover mixed-format reorder column reads#366

Open
liujiwen-up wants to merge 1 commit into
apache:mainfrom
liujiwen-up:fix/258-mixed-format-reorder-move-column
Open

test: cover mixed-format reorder column reads#366
liujiwen-up wants to merge 1 commit into
apache:mainfrom
liujiwen-up:fix/258-mixed-format-reorder-move-column

Conversation

@liujiwen-up
Copy link
Copy Markdown
Contributor

@liujiwen-up liujiwen-up commented Jun 5, 2026

Purpose

Linked issue: #258

Add focused Spark-provisioned integration coverage for mixed-format schema evolution reads after ALTER COLUMN ... FIRST/AFTER.

This PR intentionally covers only the reorder/move-column case to avoid overlapping with #354, which covers mixed-format add-column and type-promotion reads.

Brief change log

  • Add a Spark fixture with Parquet, ORC, and Avro files written across column reorder/move schema versions.
  • Add an integration test verifying full reads expose the current table schema order.
  • Verify mixed-format reads map values by field id rather than physical column position.
  • Verify projected reads preserve caller-specified projection order after column reorder/move.

Tests

  • cargo fmt: passed
  • git diff --check: passed
  • cargo test -p paimon-integration-tests --test read_tables --no-run: passed

API and Format

No API or storage format changes. This PR only adds Spark provisioning and Rust integration test coverage.

Documentation

No documentation update required.

@liujiwen-up liujiwen-up changed the title test: cover mixed-format schema evolution reads test: add mixed-format schema evolution reads Jun 5, 2026
@liujiwen-up liujiwen-up closed this Jun 5, 2026
@liujiwen-up liujiwen-up reopened this Jun 7, 2026
@liujiwen-up liujiwen-up force-pushed the fix/258-mixed-format-reorder-move-column branch from 7c64a66 to ca012be Compare June 7, 2026 15:36
@liujiwen-up liujiwen-up changed the title test: add mixed-format schema evolution reads test: cover mixed-format reorder column reads Jun 7, 2026
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.

1 participant