You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "andishgar (via GitHub)" <gi...@apache.org> on 2025/05/28 14:08:13 UTC
Re: [PR] GH-46578: [C++][Statistics] Fix the ownership handling of arrow::ArrayData::statistics in arrow::ArrayData::CopyTo and arrow::ArrayDataViewOrCopy [arrow]
andishgar commented on PR #46625:
URL: https://github.com/apache/arrow/pull/46625#issuecomment-2916496659
@kou
Before we begin the discussion, I’d like to share two notes:
1-While I’ve tried to follow the project’s coding guidelines, my primary goal was to provide an initial solution quickly to facilitate further discussion. Let’s first focus on whether the proposed solution is correct. Once we agree on that, we can refine the code structure. If anything is unclear in the code, please let me know—I’ll be happy to clarify it.
2-Currently, there are no existing tests for arrow::ArrayData::ViewOrCopyTo. This PR could also be seen as a starting point for introducing test coverage for that function.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org