You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by al...@apache.org on 2022/08/19 13:11:28 UTC

[arrow-datafusion] branch master updated: remove code that's in the arrow library now (#3206)

This is an automated email from the ASF dual-hosted git repository.

alamb pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-datafusion.git


The following commit(s) were added to refs/heads/master by this push:
     new a52dbb0de remove code that's in the arrow library now (#3206)
a52dbb0de is described below

commit a52dbb0de618cc5822a0d29a2a4ddeab9fe39786
Author: kmitchener <km...@gmail.com>
AuthorDate: Fri Aug 19 09:11:22 2022 -0400

    remove code that's in the arrow library now (#3206)
---
 datafusion/core/tests/sql/mod.rs | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/datafusion/core/tests/sql/mod.rs b/datafusion/core/tests/sql/mod.rs
index 3e50b4346..131c3c8bc 100644
--- a/datafusion/core/tests/sql/mod.rs
+++ b/datafusion/core/tests/sql/mod.rs
@@ -887,21 +887,6 @@ fn col_str(column: &ArrayRef, row_index: usize) -> String {
         return "NULL".to_string();
     }
 
-    // Special case ListArray as there is no pretty print support for it yet
-    if let DataType::FixedSizeList(_, n) = column.data_type() {
-        let array = column
-            .as_any()
-            .downcast_ref::<FixedSizeListArray>()
-            .unwrap()
-            .value(row_index);
-
-        let mut r = Vec::with_capacity(*n as usize);
-        for i in 0..*n {
-            r.push(col_str(&array, i as usize));
-        }
-        return format!("[{}]", r.join(","));
-    }
-
     array_value_to_string(column, row_index)
         .ok()
         .unwrap_or_else(|| "???".to_string())