Add new todo comment

yibin87 · yibin87 · commit 111abf32b4e6 · 2024-04-24T11:40:30.000+08:00
Signed-off-by: yibin &lt;huyibin@pingcap.com&gt;
diff --git a/pkg/planner/core/rule_eliminate_projection.go b/pkg/planner/core/rule_eliminate_projection.go
@@ -141,6 +141,7 @@ func doPhysicalProjectionElimination(p base.PhysicalPlan) base.PhysicalPlan {
 	child := p.Children()[0]
 	if childProj, ok := child.(*PhysicalProjection); ok {
 		// when current projection is an empty projection(schema pruned by column pruner), no need to reset child's schema
+		// TODO: avoid producing empty projection in column pruner.
 		if p.Schema().Len() != 0 {
 			childProj.SetSchema(p.Schema())
 		}

Original file line number	Diff line number	Diff line change
`@@ -141,6 +141,7 @@ func doPhysicalProjectionElimination(p base.PhysicalPlan) base.PhysicalPlan {`
`141`	`141`	`child := p.Children()[0]`
`142`	`142`	`if childProj, ok := child.(*PhysicalProjection); ok {`
`143`	`143`	`// when current projection is an empty projection(schema pruned by column pruner), no need to reset child's schema`
	`144`	`+ // TODO: avoid producing empty projection in column pruner.`
`144`	`145`	`if p.Schema().Len() != 0 {`
`145`	`146`	`childProj.SetSchema(p.Schema())`
`146`	`147`	`}`