Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22259#discussion_r224512333
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUDF.scala
---
@@ -47,7 +48,8 @@ case class ScalaUDF(
inputTypes: Seq[DataType] = Nil,
udfName: Option[String] = None,
nullable: Boolean = true,
- udfDeterministic: Boolean = true)
+ udfDeterministic: Boolean = true,
+ nullableTypes: Seq[Boolean] = Nil)
--- End diff --
I'm OK with that, but obviously we need to get this into 2.4 if it's going
to change. There's an RC out that's almost passed, and I flagged it. @cloud-fan
it's worth considering a new RC for a change like this, even as I'm always a
little uneasy about late significant changes. That said we need to release it
in the way we want it. (And there are a few other minor doc things we could get
into 2.4)
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
