avantgardnerio commented on code in PR #3363:
URL: https://github.com/apache/arrow-datafusion/pull/3363#discussion_r963230854
##########
datafusion/core/src/physical_plan/explain.rs:
##########
@@ -121,8 +121,8 @@ impl ExecutionPlan for ExplainExec {
)));
}
- let mut type_builder =
StringBuilder::new(self.stringified_plans.len());
- let mut plan_builder =
StringBuilder::new(self.stringified_plans.len());
+ let mut type_builder = StringBuilder::new();
Review Comment:
So always `1024` when we don't know? Lack of knowledge of a good default was
the only reason I didn't do that everywhere.
--
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-unsubscr...@arrow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
