efegokdemir opened a new pull request, #11970:
URL: https://github.com/apache/arrow-rs/pull/11970

   # Which issue does this PR close?
   - Closes #11938
   
   # Rationale for this change
   `VariantType` currently accepts every `Struct`, so callers cannot rely on it 
to reject schemas that cannot represent Variant data. This validates the 
canonical storage layout while retaining valid encoded and shredded forms.
   
   # What changes are included in this PR?
   `VariantType::try_new` and `validate` now check the required metadata field 
and the `value`/`typed_value` layout, including nested object and list shapes. 
Binary storage checks are shared with the array reader.
   
   AI disclosure: AI assistance drafted the implementation and regression tests.
   
   # Are these changes tested?
   Added regression coverage for malformed schemas and valid encoded and 
shredded layouts. All 374 `parquet-variant-compute` unit tests passed.
   
   # Are there any significant user-facing changes?
   Malformed Variant extension schemas that were previously accepted now fail 
validation.
   


-- 
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]

Reply via email to