jorisvandenbossche commented on PR #42223: URL: https://github.com/apache/arrow/pull/42223#issuecomment-2273485553
@pitrou do you have any thoughts on the API design here? - Either we simply mimic the C++ API exactly, i.e. add a `copy_to()` and `view_or_copy_to()` on Array/RecordBatch (although then on the Buffer class it is `copy()` and not `copy_to()` if we follow C++ .., which is a bit inconsistent) - or add a bit more typical python `copy()` method with an optional device argument (and default would then be to copy to the same device). But question is then if we would include the functionality of `ViewOrCopyTo` in that same method? (at which point the result could be only a "shallow" copy, i.e. same buffers in new Array/RecordBatch object) In any case we can't add a `view()` method as that is already taken on Array (for viewing as another data type) -- 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]
