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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
