Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22063#discussion_r212445066
--- 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 --
Yeah, messy. The constructor and class are public so felt it was worth
erring on the side of compatibility.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
