Skip to content

fix(variant): improve malformed object field diagnostics - #10638

Open
cakeni wants to merge 1 commit into
apache:mainfrom
cakeni:fix/variant-malformed-object-field
Open

fix(variant): improve malformed object field diagnostics#10638
cakeni wants to merge 1 commit into
apache:mainfrom
cakeni:fix/variant-malformed-object-field

Conversation

@cakeni

@cakeni cakeni commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

Which issue does this PR close?

Rationale for this change

A named child of a shredded object is required to use the shredded Variant field Struct layout. Treating a present child with another Arrow type as a missing path would hide malformed input, so the existing InvalidArgumentError is the correct boundary.

What changes are included in this PR?

  • Document the shredded-object field invariant where it is validated.
  • Include the field name and actual Arrow type in the error.
  • Add a malformed Int32 child regression test.

Are these changes tested?

  • cargo +stable-x86_64-pc-windows-gnu test -p parquet-variant-compute --lib (348 passed)
  • cargo fmt --all -- --check
  • git diff --check

Are there any user-facing changes?

Malformed shredded object errors now identify the offending field and physical type. There are no public API changes.

@github-actions github-actions Bot added the parquet-variant parquet-variant* crates label Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

parquet-variant parquet-variant* crates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Variant] Document and test validation of malformed shredded object fields

1 participant