Re: [PR] [c#] ExportedAllocationOwner should use 64-bit integer to track total allocated memory. [arrow]
github-actions[bot] commented on PR #47011:
URL: https://github.com/apache/arrow/pull/47011#issuecomment-3044885676
Thanks for opening a pull request!
If this is not a [minor
PR](https://github.com/apache/arrow/blob/main/CONTRIBUTING.md#Minor-Fixes).
Could you open an issue for this pull request on GitHub?
https://github.com/apache/arrow/issues/new/choose
Opening GitHub issues ahead of time contributes to the
[Openness](http://theapacheway.com/open/#:~:text=Openness%20allows%20new%20users%20the,must%20happen%20in%20the%20open.)
of the Apache Arrow project.
Then could you also rename the pull request title in the following format?
GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}
or
MINOR: [${COMPONENT}] ${SUMMARY}
See also:
* [Other pull requests](https://github.com/apache/arrow/pulls/)
* [Contribution Guidelines - Contributing
Overview](https://arrow.apache.org/docs/developers/overview.html)
--
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]
[PR] [c#] ExportedAllocationOwner should use 64-bit integer to track total allocated memory. [arrow]
marcin-krystianc opened a new pull request, #47011: URL: https://github.com/apache/arrow/pull/47011 ### Rationale for this change Fixes https://github.com/apache/arrow/issues/47009 ### What changes are included in this PR? `ExportedAllocationOwner` now uses a 64-bit (instead of 32-bit) variable to track total allocated memory. ### Are these changes tested? yes ### Are there any user-facing changes? **This PR contains a "Critical Fix".** Previously, it wasn't possible to export Record Batches larger than 2GB, it resulted in overflowing the integer variable that was used to track allocated memory. -- 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]
