Github user DylanGuedes commented on the issue:
https://github.com/apache/spark/pull/21045
@mgaido91 Thank you for the suggestions and for being so patient. I updated
the code with `zip` name, more tests in CollectionExpression (I'll add more
after adding support to any number of arrays as input), the stripMargin syntax
and a little refactor in the doGenCode (Is it the change that you was thinking
of? I can make the code shorter but i'll be more complicated I think).
I didn't find the proper way to add support to any number of arrays, since
the number of inputs looks limited (unary, binary and ternary expressions,
didn't find a "n-nary" or something similar). What you think?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
