davidm-db commented on code in PR #47756:
URL: https://github.com/apache/spark/pull/47756#discussion_r1738866375


##########
sql/core/src/main/scala/org/apache/spark/sql/scripting/SqlScriptingExecutionNode.scala:
##########
@@ -323,3 +323,58 @@ class WhileStatementExec(
     body.reset()
   }
 }
+
+/**
+ * Executable node for RepeatStatement.
+ * @param condition Executable node for the condition.
+ * @param body Executable node for the body.
+ * @param session Spark session that SQL script is executed within.
+ */
+class RepeatStatementExec(
+  condition: SingleStatementExec,
+  body: CompoundBodyExec,
+  session: SparkSession) extends NonLeafStatementExec {
+
+  private object RepeatState extends Enumeration {

Review Comment:
   this is a good point, but let's aim to solve this in a follow-up PR to 
simplify things a bit in this PR, since logic is correct.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to