[PR] Reject merging unions with different modes [arrow-rs]

["timmyb32r (via GitHub)" <gi...@apache.org>]

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

   # Which issue does this PR close?
   
   None — standalone bug fix.
   
   # Rationale for this change
   
   `Field::try_merge` accepted unions with different modes, even though
   sparse and dense unions have different physical layouts. The result
   retained the mode of `self`, silently ignoring the incompatible mode
   of the other field.
   
   # What changes are included in this PR?
   
   Return `ArrowError::SchemaError` when the union modes differ.
   
   # Are these changes tested?
   
   Added a regression test covering rejection of sparse/dense merges
   in both directions. It also verifies that unions with matching modes
   can still merge distinct variants.
   
   # Are there any significant user-facing changes?


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] Reject merging `Field`s of unions with different modes [arrow-rs]

["Jefffrey (via GitHub)" <gi...@apache.org>]

Jefffrey merged PR #11827:
URL: https://github.com/apache/arrow-rs/pull/11827


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org