Dandandan commented on code in PR #2690:
URL: https://github.com/apache/arrow-datafusion/pull/2690#discussion_r889492584
##########
datafusion/expr/src/logical_plan/plan.rs:
##########
@@ -656,10 +656,22 @@ impl LogicalPlan {
ref limit,
..
}) => {
+ let projected_fields = match projection {
+ Some(indices) => {
+ let schema = source.schema();
+ let names: Vec<&str> = indices
+ .iter()
+ .map(|i| schema.field(*i).name().as_str())
+ .collect();
+ format!("Some([{}])", names.join(", "))
Review Comment:
What about only showing the projection when there is one and ommiting it
when there are none.
This could remove the None/Some too:
`TableScan a projection=[col1,col2]`
vs
`TableScan a`
--
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
