miland-db commented on code in PR #52298:
URL: https://github.com/apache/spark/pull/52298#discussion_r2341228082
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala:
##########
@@ -4144,6 +4144,15 @@ object SQLConf {
.booleanConf
.createWithDefault(true)
+ val SQL_SCRIPTING_CONTINUE_HANDLER_ENABLED =
+ buildConf("spark.sql.scripting.continueHandlerEnabled")
+ .doc("SQL Scripting CONTINUE HANDLER feature is under development and
its use should be " +
+ "done under this feature flag. SQL Scripting CONTINUE HANDLER enables
users to use " +
+ "CONTINUE HANDLER syntax inside SQL Scripts.")
Review Comment:
Let's either rephrase or remove the second sentence because it doesn't bring
any value. Also we should maybe hint that this is temporary conf and it should
not be turned on/users should not use `CONTINUE HANDLERS` until they are fully
supported.
--
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
