Github user mgaido91 commented on the issue:

    https://github.com/apache/spark/pull/22131
  
    sorry, I wasn't very specific in my previous comment. I meant adding a 
method like:
    
    ```
    def validateBinding(e: HigherOrderFunction): Expression = {
        e.bind { case (f: LambdaFunction, dataTypes) =>
          f.arguments.zip(dataTypes).foreach {
            case (arg, (dt, n)) =>
              assert(arg.dataType == dt)
              assert(arg.nullable == n)
          }
          f
        }
      }
    ```
    
    and then validate all the expressions, so use it like:
    ```
    validateBinding(ArrayTransform(....))
    ```
    What do you think about this?


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to