You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "ion-elgreco (via GitHub)" <gi...@apache.org> on 2025/01/13 08:07:43 UTC

[I] `Field.try_merge` should have optional metadata merge [arrow-rs]

ion-elgreco opened a new issue, #6971:
URL: https://github.com/apache/arrow-rs/issues/6971

   **Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
   Merging fields is useful but the implicit merge of metadata is something that should be opt-in. At the moment we have to undo this to get the required effect.


-- 
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.apache.org

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


Re: [I] `Field.try_merge` should have optional metadata merge [arrow-rs]

Posted by "tustvold (via GitHub)" <gi...@apache.org>.
tustvold commented on issue #6971:
URL: https://github.com/apache/arrow-rs/issues/6971#issuecomment-2587875067

   I wonder if we really just need to provide a visitor API to allow people to define what schema merging logic they desire... We can then implement try_merge using this, and if people want something different they can implement their own visitor


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