miland-db commented on code in PR #53530:
URL: https://github.com/apache/spark/pull/53530#discussion_r2690632340
##########
sql/core/src/main/scala/org/apache/spark/sql/scripting/SqlScriptingExecutionContext.scala:
##########
@@ -244,3 +336,51 @@ class SqlScriptingExecutionScope(
errorHandler
}
}
+
+/**
+ * Defines a cursor with its name and SQL query text.
+ * This is immutable - lifecycle state is managed separately via CursorState.
+ *
+ * @param name
+ * Name of the cursor.
+ * @param queryText
Review Comment:
All other classes in this file are general sql scripting execution concepts
and are not related specific statements. This `CursorState` is there to track
the cursor state and is not even used in this file. Can we move this to another
file so we don't pollute the code too much, please?
--
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]
