Skip to content

Expose memory tracking API to Array and ArrayData #8040

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

waynexia
Copy link
Member

@waynexia waynexia commented Aug 5, 2025

Which issue does this PR close?

We generally require a GitHub issue to be filed for all bug fixes and enhancements and this helps us generate change logs for our releases. You can link an issue to this PR using the GitHub syntax.

Rationale for this change

#7303 implements the fundamental symbols for tracking memory. This patch exposes those APIs to a higher level Array and ArrayData.

What changes are included in this PR?

New APIs claim for Array and ArrayData. New feature pool to arrow, arrow-array and arrow-data for the new API

Are these changes tested?

Yes, and there is an example to demo basic usage.

Are there any user-facing changes?

New API and feature

@github-actions github-actions bot added the arrow Changes to the arrow crate label Aug 5, 2025
@waynexia waynexia force-pushed the array-mem-tracking branch from 9ef341f to 4fa363c Compare August 5, 2025 02:49
Signed-off-by: Ruihang Xia <[email protected]>
@waynexia waynexia force-pushed the array-mem-tracking branch from 31b8c72 to 27f8c46 Compare August 5, 2025 03:07
@alamb
Copy link
Contributor

alamb commented Aug 6, 2025

Thanks @waynexia -- is there a high level description / "[EPIC]" style ticket of what we are doing with these APIs? I think it would be good to make an issue that describes the high level project / direction for wider visibility (and likely to get other people involved)

I found some related issues. Do any of them reflect your plan?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
arrow Changes to the arrow crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants