Skip to content

fix: fail schema select on missing columns#759

Open
fallintoplace wants to merge 1 commit into
apache:mainfrom
fallintoplace:fix/strict-schema-select
Open

fix: fail schema select on missing columns#759
fallintoplace wants to merge 1 commit into
apache:mainfrom
fallintoplace:fix/strict-schema-select

Conversation

@fallintoplace

Copy link
Copy Markdown
Contributor

Summary

  • make Schema::Select() return an error when a requested column name cannot be resolved
  • validate TableScanBuilder::Select() during Build() so bad scan projections fail early
  • keep manifest reader projection best-effort at its documented boundary by filtering unavailable manifest/data-file columns before calling Schema::Select()
  • update schema tests and add table scan coverage for partially missing projections

Tests

  • cmake --build build --target schema_test scan_test
  • ctest --test-dir build -R 'schema_test|scan_test' --output-on-failure\n- git diff --check\n

@fallintoplace fallintoplace changed the title Fail schema select on missing columns fix: fail schema select on missing columns Jun 18, 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