[GitHub] spark pull request #14850: [SPARK-17279][SQL] better error message for NPE d...

2016-09-01 Thread clockfly
Github user clockfly commented on a diff in the pull request: https://github.com/apache/spark/pull/14850#discussion_r77184294 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUDF.scala --- @@ -1057,5 +1059,19 @@ case class ScalaUDF(

[GitHub] spark pull request #14850: [SPARK-17279][SQL] better error message for NPE d...

2016-09-01 Thread clockfly
Github user clockfly commented on a diff in the pull request: https://github.com/apache/spark/pull/14850#discussion_r77182217 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUDF.scala --- @@ -1039,9 +1032,18 @@ case class ScalaUDF( (

[GitHub] spark pull request #14850: [SPARK-17279][SQL] better error message for NPE d...

2016-09-01 Thread clockfly
Github user clockfly commented on a diff in the pull request: https://github.com/apache/spark/pull/14850#discussion_r77177520 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUDF.scala --- @@ -1057,5 +1059,19 @@ case class ScalaUDF(

[GitHub] spark pull request #14850: [SPARK-17279][SQL] better error message for NPE d...

2016-08-27 Thread yhuai
Github user yhuai commented on a diff in the pull request: https://github.com/apache/spark/pull/14850#discussion_r76518705 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/ScalaUDF.scala --- @@ -1057,5 +1057,17 @@ case class ScalaUDF( pr

[GitHub] spark pull request #14850: [SPARK-17279][SQL] better error message for NPE d...

2016-08-27 Thread cloud-fan
GitHub user cloud-fan opened a pull request: https://github.com/apache/spark/pull/14850 [SPARK-17279][SQL] better error message for NPE during ScalaUDF execution ## What changes were proposed in this pull request? When `ScalaUDF` throws NPE, it's possibly because users don't