Skip to content

[C++][Python] Remove deprecated functions in 20.0 #45680

Description

@pitrou

Describe the enhancement requested

We should review deprecated APIs and remove them if their deadline has been reached.

Component(s)

C++, Python

Activity

  1. added this to the 20.0.0 milestone on Mar 5, 2025
  2. pitrou commented on Mar 5, 2025

    @pitrou
    MemberAuthor
  3. self-assigned this
    on Mar 10, 2025
  4. AlenkaF commented on Mar 11, 2025

    @AlenkaF
    Member

    I have checked the C++ and Python parts of the codebase an found these deprecated APIs. I will start with the Python part. Would be great, in the meantime, to get some feedback on the C++ and Flight part in case I am off somewhere or I should split the work in multiple PRs for easier review.

    C++

    • GetNextSegment and related in grouper.cc from GH-44052 (Deprecated in 18.0.0)
    • GetCapacity in Gandiva cache.cc from GH-41329(Deprecated in 17.0.0)
    • BufferReader constructors in io/memory.h from GH-37212 (Deprecated in 14.0.0)
    • ObjectType, FileStatistics in io/hdfs.h from ARROW-8224 (Deprecated in 0.17.0)
    • OpenAppendStream in filesystem/gcsfs.h from ARROW-14222 (Deprecated in 6.0.0)
    • Leftover from ARROW-15732 (Deprecated in 11.0.0)

    C++ Deprecated in 19.0.0, need to wait

    • GetRecordBatchReader in reader.h from GH-44808
    • NewRowGroup with chunk_size parquet/arrow/writer.h from GH-45048

    Python C++

    Python

    Flight connected

  5. pitrou commented on Mar 11, 2025

    @pitrou
    MemberAuthor

    cc @zanmato1984 for GetNextSegment. Other C++ removals look fine to me.

  6. pitrou commented on Mar 11, 2025

    @pitrou
    MemberAuthor

    cc @lidavidm @zeroshade for Flight deprecations.

  7. zanmato1984 commented on Mar 11, 2025

    @zanmato1984
    Contributor

    The removal of GetNextSegment looks good to me. Thank you @AlenkaF for bringing this up!

  8. lidavidm commented on Mar 11, 2025

    @lidavidm
    Member

    Those removals look good to me, thanks.

  9. AlenkaF commented on Mar 28, 2025

    @AlenkaF
    Member

    I will be moving some of the deprecations still left to a new issue: #45968 so we can close this one for the 20.0.0 release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions