Skip to content

GH-51100: [C++] Deprecate IPC tensor/sparse tensor messages - #51102

Draft
pitrou wants to merge 1 commit into
apache:mainfrom
pitrou:tensor-ipc-deprecate
Draft

GH-51100: [C++] Deprecate IPC tensor/sparse tensor messages#51102
pitrou wants to merge 1 commit into
apache:mainfrom
pitrou:tensor-ipc-deprecate

Conversation

@pitrou

@pitrou pitrou commented Aug 31, 2026

Copy link
Copy Markdown
Member

Rationale for this change

Deprecate IPC APIs for reading and writing Tensor / SparseTensor messages.

Deprecating IPC support for Tensor and SparseTensor was discussed in https://lists.apache.org/thread/z2kl4g1353mbtgpt3qo7j3bzohq0q7fv.

Are these changes tested?

By existing CI tests.

Are there any user-facing changes?

No.

@github-actions

Copy link
Copy Markdown

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

@pitrou

pitrou commented Aug 31, 2026

Copy link
Copy Markdown
Member Author

Let's keep this PR draft until final consensus is reached on the ML.

@rok

rok commented Aug 31, 2026

Copy link
Copy Markdown
Member

@pitrou I'm very much on board with the sparse tensors. I so wonder about tensors, are we sure this is not being used? I'm ok to add a deprecation and wait for a release to see if we hear back. But my expectation is there would be some users out there.

@pitrou

pitrou commented Aug 31, 2026

Copy link
Copy Markdown
Member Author

I'm ok to add a deprecation and wait for a release to see if we hear back. But my expectation is there would be some users out there.

Hopefully the deprecation will have them chime in? But my attempt at code-searching this on GH did not lead to many recent relevant results.

@rok

rok commented Aug 31, 2026

Copy link
Copy Markdown
Member

I'm ok to add a deprecation and wait for a release to see if we hear back. But my expectation is there would be some users out there.

Hopefully the deprecation will have them chime in? But my attempt at code-searching this on GH did not lead to many recent relevant results.

Agreed. I'll take a look myself and report back.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants