add TODO comment

jorisvandenbossche · Jun 26, 2024 · fe82b3e · fe82b3e
1 parent 1b64ccf
commit fe82b3e
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/cpp/src/arrow/pretty_print.cc b/cpp/src/arrow/pretty_print.cc
@@ -392,6 +392,8 @@ class ArrayPrinter : public PrettyPrinter {
 
   Status Print(const Array& array) {
     if (array.device_type() != DeviceAllocationType::kCPU) {
+      // TODO: ideally we only copy start/end slice based on the window size that is being
+      // printed (requires CopyTo for slices https://github.com/apache/arrow/issues/43055)
       ARROW_ASSIGN_OR_RAISE(auto array_cpu, array.CopyTo(default_cpu_memory_manager()));
       RETURN_NOT_OK(VisitArrayInline(*array_cpu, this));
     } else {