cloud-fan commented on a change in pull request #30173:
URL: https://github.com/apache/spark/pull/30173#discussion_r513651626
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/QueryPlan.scala
##########
@@ -180,10 +180,14 @@ abstract class QueryPlan[PlanType <: QueryPlan[PlanType]]
extends TreeNode[PlanT
* rewrite attribute references in the parent nodes.
* @param skipCond a boolean condition to indicate if we can skip
transforming a plan node to save
* time.
+ * @param canGetOutput a boolean condition to indicate if we can get the
output of a plan node
+ * to prune the attributes mapping to be propagated. The
default value is true
+ * as only unresolved logical plan can't get output.
Review comment:
One advantage of the current way is we can skip the check for optimizer
rules (use the default value true).
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
