You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "tustvold (via GitHub)" <gi...@apache.org> on 2023/04/15 10:37:57 UTC

[GitHub] [arrow-rs] tustvold commented on pull request #4068: Add DictionaryArray Constructors (#3879)

tustvold commented on PR #4068:
URL: https://github.com/apache/arrow-rs/pull/4068#issuecomment-1509722583

   IMO marking it deprecated and removing it is only meaningful if there is a tangible behaviour change that is liable to cause people to defer making the change. As it stands, changing the arguments to be owned is likely as much a change as adding `allow(deprecated)]`.
   
   My vote is just to get it over and done with, but I will leave this open to see if anybody else weighs in


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