[GitHub] [arrow-datafusion] comphead commented on a diff in pull request #6359: Limit the number of partition files to be displayed for FileGroupsDisplay

["comphead (via GitHub)" <gi...@apache.org>]

comphead commented on code in PR #6359:
URL: https://github.com/apache/arrow-datafusion/pull/6359#discussion_r1195339879


##########
datafusion/core/src/physical_plan/file_format/mod.rs:
##########
@@ -275,7 +286,9 @@ impl<'a> Display for FileGroupsDisplay<'a> {
         let mut first_group = true;
         let groups = if self.0.len() == 1 { "group" } else { "groups" };
         write!(f, "{{{} {}: [", self.0.len(), groups)?;
-        for group in self.0 {
+        // To avoid showing too many partitions

Review Comment:
   thats nice, thanks.



-- 
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.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org