Don't include length and offset when pretty-printing REE

apache · Jan 23, 2023 · 4be0ed1 · 4be0ed1
1 parent 5f9cee8
commit 4be0ed1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 9 deletions.
diff --git a/cpp/src/arrow/array/array_run_end_test.cc b/cpp/src/arrow/array/array_run_end_test.cc
@@ -471,7 +471,7 @@ TEST_P(TestRunEndEncodedArray, Printing) {
   ASSERT_OK(PrettyPrint(*int_array, {}, &ss));
   ASSERT_EQ(ss.str(),
             "\n"
-            "-- run ends array (offset: 0, logical length: 30)\n"
+            "-- run_ends:\n"
             "  [\n"
             "    10,\n"
             "    20,\n"
@@ -490,7 +490,7 @@ TEST_P(TestRunEndEncodedArray, Printing) {
   ASSERT_OK(PrettyPrint(*string_array, {}, &ss));
   ASSERT_EQ(ss.str(),
             "\n"
-            "-- run ends array (offset: 0, logical length: 30)\n"
+            "-- run_ends:\n"
             "  [\n"
             "    10,\n"
             "    20,\n"
@@ -508,7 +508,7 @@ TEST_P(TestRunEndEncodedArray, Printing) {
   ASSERT_OK(PrettyPrint(*sliced_array, {}, &ss));
   ASSERT_EQ(ss.str(),
             "\n"
-            "-- run ends array (offset: 15, logical length: 6)\n"
+            "-- run_ends:\n"
             "  [\n"
             "    10,\n"
             "    20,\n"
@@ -529,7 +529,7 @@ TEST_P(TestRunEndEncodedArray, Printing) {
   ASSERT_OK(PrettyPrint(*empty_array, {}, &ss));
   ASSERT_EQ(ss.str(),
             "\n"
-            "-- run ends array (offset: 0, logical length: 0)\n"
+            "-- run_ends:\n"
             "  []\n"
             "-- values:\n"
             "  []");

diff --git a/cpp/src/arrow/pretty_print.cc b/cpp/src/arrow/pretty_print.cc
@@ -380,11 +380,7 @@ class ArrayPrinter : public PrettyPrinter {
   Status Visit(const RunEndEncodedArray& array) {
     Newline();
     Indent();
-    Write("-- run ends array (offset: ");
-    Write(std::to_string(array.offset()));
-    Write(", logical length: ");
-    Write(std::to_string(array.length()));
-    Write(")\n");
+    Write("-- run_ends:\n");
     RETURN_NOT_OK(PrettyPrint(*array.run_ends_array(), ChildOptions(true), sink_));
 
     Newline();