learningchess2003 commented on code in PR #41864:
URL: https://github.com/apache/spark/pull/41864#discussion_r1256520869
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/generators.scala:
##########
@@ -431,6 +432,11 @@ case class Explode(child: Expression) extends ExplodeBase {
copy(child = newChild)
}
+object Explode extends SupportsNamedArguments {
Review Comment:
Right now, we don't really have support for named arguments for final
variadic argument. We can actually include support for it if needed. I think it
just probably requires a bit of extra logic to handle the case. @dtenedor What
do you think?
--
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
