Skip to content

fix: blob projection schema compatibility#6521

Merged
Xuanwo merged 3 commits intomainfrom
xuanwo/fix-blob-projection-schema
Apr 15, 2026
Merged

fix: blob projection schema compatibility#6521
Xuanwo merged 3 commits intomainfrom
xuanwo/fix-blob-projection-schema

Conversation

@Xuanwo
Copy link
Copy Markdown
Collaborator

@Xuanwo Xuanwo commented Apr 15, 2026

Blob columns can be represented either as loaded values or as unloaded descriptor schemas, but our schema projection logic still treated those views as incompatible types. This change teaches field projection and intersection to recognize blob loaded/unloaded pairs as the same logical column, and adds regression coverage for both the core schema path and the projection-plan path that previously failed.

@github-actions
Copy link
Copy Markdown
Contributor

ACTION NEEDED
Lance follows the Conventional Commits specification for release automation.

The PR title and description are used as the merge commit message. Please update your PR title and description to match the specification.

For details on the error please inspect the "PR Title Check" action.

@Xuanwo Xuanwo changed the title Fix blob projection schema compatibility fix: blob projection schema compatibility Apr 15, 2026
@github-actions github-actions Bot added the bug Something isn't working label Apr 15, 2026
@Xuanwo Xuanwo marked this pull request as ready for review April 15, 2026 06:27
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 15, 2026

Codecov Report

❌ Patch coverage is 96.66667% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
rust/lance-core/src/datatypes/field.rs 96.42% 1 Missing ⚠️
rust/lance/src/dataset/take.rs 96.87% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@Xuanwo Xuanwo merged commit 2a4ed6c into main Apr 15, 2026
29 checks passed
@Xuanwo Xuanwo deleted the xuanwo/fix-blob-projection-schema branch April 15, 2026 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants