-
Notifications
You must be signed in to change notification settings - Fork 878
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove APIs deprecated since v 4.4.0 #6722
Remove APIs deprecated since v 4.4.0 #6722
Conversation
The deprecated APIs are not supposed to stay around for ever. The purpose of deprecation is to eventually remove.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @findepi for the cleanup work.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @findepi
Main is open for breaking changes now that 53.3.0 has been released so 🚀 |
Thanks again @findepi |
thanks for the merge!
is #6609 good to go too? or any other flagged with |
Yup -- I just haven't had a chance to move them along. I plan to devote some time this afternoon to moving arrow-rs along |
Which issue does this PR close?
None
Rationale for this change
The deprecated APIs are not supposed to stay around for ever. The purpose of deprecation is to eventually remove.
What changes are included in this PR?
Remove
flight_schema_from_arrow_schema
,flight_data_from_arrow_schema
,ipc_message_from_arrow_schema
, that were deprecated for about three and a half years.Are there any user-facing changes?
yes