Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22063#discussion_r212499164
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUDF.scala
 ---
    @@ -57,8 +59,21 @@ case class ScalaUDF(
           children: Seq[Expression],
           inputTypes: Seq[DataType],
           udfName: Option[String]) = {
    -    this(
    -      function, dataType, children, inputTypes, udfName, nullable = true, 
udfDeterministic = true)
    +    this(function, dataType, children, inputTypes, udfName,
    +      nullable = true, udfDeterministic = true, nullableTypes = Nil)
    +  }
    +
    +  // Constructor from Spark 2.3
    --- End diff --
    
    By convention, everything under catalyst package is private, so 
compatibility is not a concern here.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to