Github user stanzhai commented on a diff in the pull request:

    https://github.com/apache/spark/pull/18544#discussion_r219485843
  
    --- Diff: sql/hive/src/test/scala/org/apache/spark/sql/hive/UDFSuite.scala 
---
    @@ -193,4 +193,29 @@ class UDFSuite
           }
         }
       }
    +
    +  test("SPARK-21318: The correct exception message should be thrown " +
    +    "if a UDF/UDAF has already been registered") {
    +    val UDAFName = "empty"
    +    val UDAFClassName = 
classOf[org.apache.spark.sql.hive.execution.UDAFEmpty].getCanonicalName
    +
    +    withTempDatabase { dbName =>
    --- End diff --
    
    @cloud-fan I just copied and modified the code from another test case, the 
default database works well.
    
    The test case has been simplified now.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to