-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[C++] Remove deprecated APIs #36326
Comments
@benibus Would you perhaps like to tackle this? |
I am moving this to 14.0.0 at the moment, if it is merged in time we can add it to the 13.0.0 but I don't think it is a blocker. |
wgtmac
pushed a commit
that referenced
this issue
Jul 19, 2023
### Rationale for this change General cleanup of C++ APIs deprecated for at least a year. ### What changes are included in this PR? Removes any APIs annotated with `ARROW_DEPRECATED` and an included version `<=9.0`. In a few cases, tests needed to be removed or slightly altered. ### Are these changes tested? Yes (covered by existing tests) when applicable. ### Are there any user-facing changes? Yes * Closes: #36326 Authored-by: benibus <bpharks@gmx.com> Signed-off-by: Gang Wu <ustcwg@gmail.com>
chelseajonesr
pushed a commit
to chelseajonesr/arrow
that referenced
this issue
Jul 20, 2023
…he#36675) ### Rationale for this change General cleanup of C++ APIs deprecated for at least a year. ### What changes are included in this PR? Removes any APIs annotated with `ARROW_DEPRECATED` and an included version `<=9.0`. In a few cases, tests needed to be removed or slightly altered. ### Are these changes tested? Yes (covered by existing tests) when applicable. ### Are there any user-facing changes? Yes * Closes: apache#36326 Authored-by: benibus <bpharks@gmx.com> Signed-off-by: Gang Wu <ustcwg@gmail.com>
R-JunmingChen
pushed a commit
to R-JunmingChen/arrow
that referenced
this issue
Aug 20, 2023
…he#36675) ### Rationale for this change General cleanup of C++ APIs deprecated for at least a year. ### What changes are included in this PR? Removes any APIs annotated with `ARROW_DEPRECATED` and an included version `<=9.0`. In a few cases, tests needed to be removed or slightly altered. ### Are these changes tested? Yes (covered by existing tests) when applicable. ### Are there any user-facing changes? Yes * Closes: apache#36326 Authored-by: benibus <bpharks@gmx.com> Signed-off-by: Gang Wu <ustcwg@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the enhancement requested
A number of C++ APIs are marked deprecated. We don't have a formal deprecation mechanism, but we should probably remove those that have been deprecated for at least one year, i.e. in 9.0.0 or before.
Component(s)
C++
The text was updated successfully, but these errors were encountered: