davidm-db commented on code in PR #51595:
URL: https://github.com/apache/spark/pull/51595#discussion_r2222763928
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/errors/SqlScriptingErrors.scala:
##########
@@ -30,10 +30,21 @@ import org.apache.spark.sql.internal.SQLConf
*/
private[sql] object SqlScriptingErrors {
+ def forVariableNameAlreadyExistsAsLabelInScope(
+ origin: Origin,
+ varName: String): Throwable = {
Review Comment:
nit: usually if the line is not longer than 100 chars, we don't split it,
but it's not that important (do whatever you like). What's important is when we
do split, we add two more spaces for indentation - check similar examples below.
--
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]
