Repository navigation
[C++] Use ViewOrCopyTo instead of CopyTo when pretty printing non-CPU data #43507
Description
As a follow-up on #41664 (PR #42010), which ensured to copy data to the CPU when trying to pretty print it.
But, the PR was using CopyTo when the data was not using a CPU device allocation type, i.e.:
if (array.device_type() != DeviceAllocationType::kCPU) {
ARROW_ASSIGN_OR_RAISE(auto array_cpu, array.CopyTo(default_cpu_memory_manager()));
...But this should better use ViewOrCopyTo, in case the data can be viewed as CPU data without a copy.
Activity
apacheGH-43507: [C++] Use ViewOrCopyTo instead of CopyTo when pretty …
GH-43507: [C++] Use ViewOrCopyTo instead of CopyTo when pretty printi…
kou commentedon Aug 2, 2024
Issue resolved by pull request 43508
#43508