GH-49356: [C++] Remove deprecated APIs from v13.0.0 and v18.0.0 by AliRana30 · Pull Request #49171 · apache/arrow

AliRana30 · 2026-02-07T09:43:24Z

Rationale for This Change

This PR removes C++ APIs that have been deprecated for more than one year, in line with Arrow's deprecation and cleanup policy. All removed APIs were deprecated before January 1, 2025.

What Changes Are Included in This PR?

This PR removes 3 deprecated C++ APIs:

1. decimal(precision, scale) function

Deprecated in v18.0.0 (September 30, 2024, PR GH-43956: [C++][Format] Add initial Decimal32/Decimal64 implementations #43957)
Replacement: smallest_decimal(precision, scale)
Removed from: cpp/src/arrow/type_fwd.h, cpp/src/arrow/type.cc

2. cuda::DefaultMemoryMapper(device_type, device_id) function

Deprecated in v16.0.0 (April 16, 2024, PR GH-40698: [C++] Create registry for Devices to map DeviceType to MemoryManager in C Device Data import #40699)
Replacement: arrow::DefaultDeviceMapper
Removed from: cpp/src/arrow/gpu/cuda_memory.h, cpp/src/arrow/gpu/cuda_memory.cc

3. HasValidityBitmap(Type::type id) global function

Deprecated in v17.0.0 (July 11, 2024, PR GH-41114: [C++] Add is_validity_defined_by_bitmap() predicate #41115)
Replacement: may_have_validity_bitmap(Type::type id)
Removed from: cpp/src/arrow/type.h
Note: Member functions like ArrayData::HasValidityBitmap() remain unchanged

Are These Changes Tested?

Yes. Verified via full codebase search that all removed symbols have zero remaining usages in the C++ codebase.

Are There Any User-Facing Changes?

Yes. Downstream C++ consumers using these deprecated APIs will need to migrate to the replacement APIs listed above.

GitHub Issue: [C++] Remove deprecated APIs from v16-v18 releases #49356

github-actions · 2026-02-07T09:43:50Z

⚠️ GitHub issue #49153 has been automatically assigned in GitHub to PR creator.

rok · 2026-02-07T11:16:32Z

This PR looks ok, however test looks AI generated so I checked the statement. It turns out decimal was not deprecated "over 6 years ago" but rather in November 2024 when 18.1.0 was released. PR for deprecation was merged on 4th September 2024.

Since I don't want to search for every claim you make here, please do the following:

expand this PR to all C++ symbols you want to deprecate and rename it appropriately
~~provide PR links where deprecation notice was given (per symbol) and the release where this commit was then released~~
Manually check which release happened a year ago and only remove symbols with deprecation notices for releases equal of less than the year old release

~~e.g.:~~

~~decimal was deprecated on 4th September 2024 which was a part of 18.1.0 (November 2024).~~

AliRana30 · 2026-02-08T16:31:11Z

@rok You're absolutely right about the dates. I apologize - decimal() was deprecated in September 2024 (only 4 months ago), not "over 6 years ago" as I mistakenly claimed.

I'll commit the changes with properly verified deprecations >1 year old (before 2025) with PR links and release dates as you requested.

Note : There's no option for renaming the PR , should I close this and create a new one):

rok · 2026-02-08T19:57:19Z

@AliRana30 you can rename the PR - there is an edit button at the top right of the page.

kou · 2026-02-19T09:30:13Z

dev/archery/archery/integration/runner.py

            "flight_sql:extension",
            description="Ensure Flight SQL extensions work as expected.",
-            skip_testers={"Rust", ".NET"}
+            skip_testers={"Rust", ".NET", "C++"}


Why do we need this?

The flight_sql:extension scenario tests the CancelQuery action, which this PR removes from C++ (it was deprecated in v13.0.0, August 2023).

C++ now uses [CancelFlightInfo] instead, which is the replacement API. However, other languages (Java, Go) still implement the old CancelQuery action.

Without this skip, the integration tests fail with:
"Action not implemented: CancelQuery rpc error: code = Unimplemented desc = Action not implemented: CancelQuery"

Since C++ no longer supports the deprecated CancelQuery action being tested in this scenario, it needs to be skipped from cross-language testing for this specific scenario.

Can we update the Java and Go implementations instead of skipping C++ tests? If we skip the C++ implementation, non CancelQuery tests are also skipped, right?

You're right - skipping C++ would skip all extension tests, not just CancelQuery.

Looking at the code, the C++ integration test already uses [CancelFlightInfo] (the replacement API). The failures were happening because the integration test servers in Java/Go still expect the CancelQuery action.

Would you prefer:

**1.**I remove CancelQuery from Java/Go integration test servers as well (they also deprecated it), OR
**2.**I keep the C++ skip for now, and we can remove CancelQuery from all languages in a coordinated PR?

Option 1 would make this PR more complete, but touches Java/Go code beyond just C++.

Option 1 is better.
We need to work on https://github.com/apache/arrow-java for Java and https://github.com/apache/arrow-go for Go.

Got it. Since Java and Go are in separate repos, should I:

1.Wait for those to be merged
2.Then update this PR to remove the C++ skip

Or would you prefer to keep the C++ skip in this PR for now, with a TODO comment that it can be removed once arrow-java and arrow-go are updated?

We should create the PRs to remove CancelQuery on Arrow Java and Arrow Go and hold this PR until this is done. Skipping the whole flight sql integration tests could introduce integration issues without us noticing. You could also split the deprecation of CancelQuery to a follow up PR until the other PRs are merged (possibly add a TODO explaining why it can't be deprecated yet) and just push the other deprecation removals. Both approaches seem reasonable.

1.Wait for those to be merged

Are there already PRs in apache/arrow-java and apache/arrow-go to migrate to CancelFlightInfo from CancelQuery?

If you want to complete this PR as soon as possible, I prefer the following step:

Remove CancelQuery related change from this PR

Merge this PR

Open a PR to apache/arrow-java to migrate to CancelFlightInfo from CancelQuery

Merge the PR

Open a PR to apache/arrow-go to migrate to CancelFlightInfo from CancelQuery

Merge the PR

Open a separated PR to this repository to migrate to CancelFlightInfo from CancelQuery

Merge the PR

If you don't mind this PR keeps opening, I prefer the following step:

Open a PR to apache/arrow-java to migrate to CancelFlightInfo from CancelQuery

Merge the PR

Open a PR to apache/arrow-go to migrate to CancelFlightInfo from CancelQuery

Merge the PR

Remove the C++ skip from this PR

Merge this PR

(Wow. Raúl also said the same approaches. :-)

Thanks for the guidance! I'll go with option of removing CancelQuery from this PR to get the other deprecations merged faster. Because this PR is purely for only those deprecations that are linked with C++ only.

I'll remove the CancelQuery changes and revert the integration test skip until this PR , so it gets merged and does'nt create mess of the tests and changes.

kou · 2026-02-21T01:37:02Z

Could you open a new issue for the CancelQuery work?

Could you update the PR description?

kou · 2026-02-21T01:38:55Z

BTW, does this cover GH-49153 entirely? If this focuses only some parts of GH-49153, we should open a new issue for this instead of using GH-49153. If we merge this PR that is associated with GH-49153, GH-49153 will be closed.

AliRana30 · 2026-02-21T11:32:19Z

@kou I've made the following changes:

Created a new issue [C++] Remove deprecated APIs from v16-v18 releases #49356 specifically for these 3 C++ API removals
Updated this PR to reference [C++] Remove deprecated APIs from v16-v18 releases #49356 instead of [C++][Python] Act on existing deprecations #49153
Removed all CancelQuery-related changes from this PR (will create a separate issue for that cross-repo work)
Updated the PR description

This PR now focuses only on:

decimal() function (v18.0.0)
cuda::DefaultMemoryMapper() (v16.0.0)
HasValidityBitmap() global function (v17.0.0)

Remove 3 deprecated C++ APIs: - decimal() function (v18.0.0, PR apache#43957) - cuda::DefaultMemoryMapper() (v16.0.0, PR apache#40699) - HasValidityBitmap() global function (v17.0.0, PR apache#41115) All deprecated before 2025.

github-actions · 2026-02-21T22:20:59Z

⚠️ GitHub issue #49356 has been automatically assigned in GitHub to PR creator.

github-actions · 2026-02-21T22:20:59Z

⚠️ GitHub issue #49356 has no components, please add labels for components.

kou

+1

github-actions bot added Component: C++ awaiting review Awaiting review labels Feb 7, 2026

AliRana30 closed this Feb 8, 2026

AliRana30 reopened this Feb 8, 2026

AliRana30 force-pushed the remove-decimal-deprecation branch from 031fe14 to 474f7c4 Compare February 18, 2026 20:07

AliRana30 requested a review from lidavidm as a code owner February 18, 2026 20:07

github-actions bot added the Component: FlightRPC label Feb 18, 2026

AliRana30 changed the title ~~GH-49153: [C++] Remove deprecated decimal() factory function~~ GH-49153: [C++] Remove deprecated APIs from v13.0.0 and v18.0.0 Feb 18, 2026

github-actions bot added the Component: Documentation label Feb 18, 2026

AliRana30 force-pushed the remove-decimal-deprecation branch from b95061c to e49383d Compare February 18, 2026 20:13

github-actions bot removed the Component: Documentation label Feb 18, 2026

AliRana30 force-pushed the remove-decimal-deprecation branch 2 times, most recently from e91ef90 to ded7299 Compare February 18, 2026 21:54

AliRana30 requested review from assignUser, jonkeane, kou and raulcd as code owners February 19, 2026 09:21

AliRana30 force-pushed the remove-decimal-deprecation branch from 73d9533 to 678b8bd Compare February 19, 2026 09:23

kou reviewed Feb 19, 2026

View reviewed changes

github-actions bot added awaiting changes Awaiting changes and removed awaiting review Awaiting review labels Feb 19, 2026

AliRana30 requested a review from kou February 19, 2026 12:04

AliRana30 force-pushed the remove-decimal-deprecation branch from 678b8bd to 01d94dd Compare February 20, 2026 17:21

github-actions bot added awaiting change review Awaiting change review and removed Component: FlightRPC awaiting changes Awaiting changes labels Feb 20, 2026

kou added the CI: Extra: CUDA Run extra CUDA CI label Feb 21, 2026

AliRana30 force-pushed the remove-decimal-deprecation branch from 01d94dd to 68262e8 Compare February 21, 2026 11:34

kou changed the title ~~GH-49153: [C++] Remove deprecated APIs from v13.0.0 and v18.0.0~~ GH-49356: [C++] Remove deprecated APIs from v13.0.0 and v18.0.0 Feb 21, 2026

kou approved these changes Feb 21, 2026

View reviewed changes

github-actions bot added awaiting merge Awaiting merge and removed awaiting change review Awaiting change review labels Feb 21, 2026

kou merged commit 8c27898 into apache:main Feb 21, 2026
62 of 66 checks passed

kou removed the awaiting merge Awaiting merge label Feb 21, 2026

kou mentioned this pull request Feb 21, 2026

[C++] Remove deprecated APIs from v16-v18 releases #49356

Closed

Comments

Conversation

AliRana30 commented Feb 7, 2026 • edited by kou Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Rationale for This Change

What Changes Are Included in This PR?

Are These Changes Tested?

Are There Any User-Facing Changes?

Uh oh!

github-actions bot commented Feb 7, 2026

Uh oh!

rok commented Feb 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

AliRana30 commented Feb 8, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rok commented Feb 8, 2026

Uh oh!

kou Feb 19, 2026

Choose a reason for hiding this comment

Uh oh!

AliRana30 Feb 19, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

kou Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

AliRana30 Feb 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

kou Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

AliRana30 Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

raulcd Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

kou Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

kou Feb 20, 2026

Choose a reason for hiding this comment

Uh oh!

AliRana30 Feb 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

kou commented Feb 21, 2026

Uh oh!

kou commented Feb 21, 2026

Uh oh!

AliRana30 commented Feb 21, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Feb 21, 2026

Uh oh!

github-actions bot commented Feb 21, 2026

Uh oh!

kou left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

AliRana30 commented Feb 7, 2026 •

edited by kou

Loading

rok commented Feb 7, 2026 •

edited

Loading

AliRana30 commented Feb 8, 2026 •

edited

Loading

AliRana30 Feb 19, 2026 •

edited

Loading

AliRana30 Feb 20, 2026 •

edited

Loading

AliRana30 Feb 20, 2026 •

edited

Loading

AliRana30 commented Feb 21, 2026 •

edited

Loading