cloud-fan commented on code in PR #49571: URL: https://github.com/apache/spark/pull/49571#discussion_r1922492638
########## sql/core/src/main/scala/org/apache/spark/sql/execution/basicPhysicalOperators.scala: ########## @@ -714,6 +718,140 @@ case class UnionExec(children: Seq[SparkPlan]) extends SparkPlan { copy(children = newChildren) } +/** + * The physical node for recursion. Currently only UNION ALL case is supported. + * + * @param loopId The id of the loop. + * @param anchor The logical plan of the initial element of the loop. + * @param recursion The logical plan that describes the recursion with an [[UnionLoopRef]] node. + * @param output The output attributes of this loop. + * @param limit An optional limit that can be pushed down to the node to stop the loop earlier. + */ +case class UnionLoopExec( + loopId: Long, + @transient anchor: LogicalPlan, + @transient recursion: LogicalPlan, + override val output: Seq[Attribute], + limit: Option[Int] = None) extends LeafExecNode { Review Comment: I need a bit more context to understand how LIMIT is applied for recursive CTE. So every loop iteration produce some results, and the loop ends when the current iteration returns no result? And with a LIMIT we can end the loop earlier? -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org