address review comments

Author: mgaido91

core/src/main/scala/org/apache/spark/api/python/PythonRunner.scala

@@ -39,6 +39,13 @@ private[spark] object PythonEvalType {
 
   val SQL_PANDAS_SCALAR_UDF = 200
   val SQL_PANDAS_GROUP_MAP_UDF = 201
+
+  def toString(pythonEvalType: Int): String = pythonEvalType match {
+    case NON_UDF => "NON_UDF"
+    case SQL_BATCHED_UDF => "SQL_BATCHED_UDF"
+    case SQL_PANDAS_SCALAR_UDF => "SQL_PANDAS_SCALAR_UDF"
+    case SQL_PANDAS_GROUP_MAP_UDF => "SQL_PANDAS_GROUP_MAP_UDF"
+  }
 }
 
 /**

python/pyspark/sql/tests.py

@@ -439,7 +439,6 @@ def test_nondeterministic_udf(self):
         import random
         udf_random_col = udf(lambda: int(100 * random.random()), IntegerType()).asNondeterministic()
         df = self.spark.createDataFrame([Row(1)]).select(udf_random_col().alias('RAND'))
-        random.seed(1234)
         udf_add_ten = udf(lambda rand: rand + 10, IntegerType())
         [row] = df.withColumn('RAND_PLUS_TEN', udf_add_ten('RAND')).collect()
         self.assertEqual(row[0] + 10, row[1])

python/pyspark/sql/udf.py

@@ -165,6 +165,8 @@ def wrapper(*args):
     def asNondeterministic(self):
         """
         Updates UserDefinedFunction to nondeterministic.
+
+        .. versionadded:: 2.3
         """
         self._deterministic = False
         return self

sql/core/src/main/scala/org/apache/spark/sql/UDFRegistration.scala

@@ -23,6 +23,7 @@ import scala.reflect.runtime.universe.TypeTag
 import scala.util.Try
 
 import org.apache.spark.annotation.InterfaceStability
+import org.apache.spark.api.python.PythonEvalType
 import org.apache.spark.internal.Logging
 import org.apache.spark.sql.api.java._
 import org.apache.spark.sql.catalyst.{JavaTypeInference, ScalaReflection}
@@ -58,6 +59,7 @@ class UDFRegistration private[sql] (functionRegistry: FunctionRegistry) extends
         | pythonIncludes: ${udf.func.pythonIncludes}
         | pythonExec: ${udf.func.pythonExec}
         | dataType: ${udf.dataType}
+        | pythonEvalType: ${PythonEvalType.toString(udf.pythonEvalType)}
         | udfDeterministic: ${udf.udfDeterministic}
       """.stripMargin)