AliRana30 commented on issue #49153: URL: https://github.com/apache/arrow/issues/49153#issuecomment-3854082662
Hi @pitrou, I've analyzed the C++ codebase and found **7 groups of deprecated APIs** from v13.0.0 to v24.0.0. ## High Priority Removals (3+ years old): 1. `HasValidityBitmap()` (v17.0.0) - 4 usages in IPC module 2. [decimal()](cci:1://file:///c:/Users/PMLS/Desktop/apache/arrow/cpp/src/arrow/type_fwd.h:541:0-542:68) factory (v18.0.0) - **No usages found**, can remove immediately 3. [GetRecordBatchReader()](cci:1://file:///c:/Users/PMLS/Desktop/apache/arrow/cpp/src/parquet/arrow/reader.cc:1005:0-1094:1) Status overloads (v21.0.0) - 3 overloads in parquet 4. `ServerAuthHandler::IsValid()` old signature (v13.0.0) - Flight module Also found medium/lower priority items in v22-v24. ## Proposed Plan: I'll tackle this in **separate PRs by version group** (v13-v21, v22-v23, v24) for easier review. All follow the same pattern: `Status Method(..., Type* out)` → `Result<Type> Method(...)` **Questions:** - Should I start with v13-v21 (oldest), or focus on specific versions? - Any concerns about breaking changes for the oldest APIs? Ready to start if this approach works! -- 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]
