trailer

Signed-off-by: Robert Kruszewski <github@robertk.io>

Author: robert3005

vortex-array/src/display/mod.rs

@@ -480,23 +480,31 @@ impl dyn DynArray + '_ {
                 write!(f, "{}", if f.alternate() { "[\n" } else { "[" })?;
                 let sep = if *omit_comma_after_space { "," } else { ", " };
                 let sep = if f.alternate() { ",\n" } else { sep };
-                let limit = std::cmp::min(self.len(), f.precision().unwrap_or(DISPLAY_LIMIT));
+                let limit = self.len().min(f.precision().unwrap_or(DISPLAY_LIMIT));
+                let is_truncated = self.len() > limit;
                 write!(
                     f,
                     "{}",
-                    (0..limit)
+                    (0..(limit - if is_truncated { 3 } else { 0 }))
                         .map(|i| self
                             .scalar_at(i)
                             .map_or_else(|e| format!("<error: {e}>"), |s| s.to_string()))
                         .format(sep)
                 )?;
-                let ellipsis = if self.len() > limit {
-                    format!("{sep}...")
-                } else {
-                    "".to_string()
-                };
+                if is_truncated {
+                    write!(f, "{sep}...{sep}")?;
+                    write!(
+                        f,
+                        "{}",
+                        (self.len() - 3..self.len())
+                            .map(|i| self
+                                .scalar_at(i)
+                                .map_or_else(|e| format!("<error: {e}>"), |s| s.to_string()))
+                            .format(sep)
+                    )?;
+                }
                 let closing_brace = if f.alternate() { "\n]" } else { "]" };
-                write!(f, "{ellipsis}{closing_brace}",)
+                write!(f, "{closing_brace}")
             }
             DisplayOptions::TreeDisplay {
                 buffers,
@@ -610,7 +618,7 @@ mod test {
         .into_array();
         assert_eq!(
             x.display_values().to_string(),
-            "[0i32, 1i32, 2i32, 3i32, 4i32, 5i32, 6i32, 7i32, 8i32, 9i32, 10i32, 11i32, 12i32, 13i32, 14i32, 15i32, ...]"
+            "[0i32, 1i32, 2i32, 3i32, 4i32, 5i32, 6i32, 7i32, 8i32, 9i32, 10i32, 11i32, 12i32, ..., 14i32, 15i32, 16i32]"
         );
     }