andygrove commented on a change in pull request #7035:
URL: https://github.com/apache/arrow/pull/7035#discussion_r418346377
##########
File path: rust/arrow/src/util/pretty.rs
##########
@@ -27,18 +27,18 @@ use prettytable::{Cell, Row, Table};
use crate::error::{ArrowError, Result};
///! Create a visual representation of record batches
-pub fn pretty_format_batches(results: &Vec<RecordBatch>) -> Result<String> {
Review comment:
I agree. I got into a bad habit of using &Vec instead of slice early on
and have been meaning to start fixing this.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
