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

    https://github.com/apache/spark/pull/8368#discussion_r37755900
  
    --- Diff: 
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2Suites.scala
 ---
    @@ -377,6 +377,60 @@ class HiveThriftBinaryServerSuite extends 
HiveThriftJdbcTest {
           rs2.close()
         }
       }
    +
    +  test("test add jar") {
    +    withMultipleConnectionJdbcStatement(
    +      {
    +        statement =>
    +          val jarFile =
    +            "../hive/src/test/resources/hive-hcatalog-core-0.13.1.jar"
    +              .split("/")
    +              .mkString(File.separator)
    +
    +          statement.executeQuery(s"ADD JAR $jarFile")
    +      },
    +
    +      {
    +        statement =>
    +          val queries = Seq(
    +            "DROP TABLE IF EXISTS smallKV",
    +            "CREATE TABLE smallKV(key INT, val STRING)",
    +            s"LOAD DATA LOCAL INPATH '${TestData.smallKv}' OVERWRITE INTO 
TABLE smallKV",
    +            "DROP TABLE IF EXISTS addJar",
    +            """CREATE TABLE addJar(key string)
    +              |ROW FORMAT SERDE 'org.apache.hive.hcatalog.data.JsonSerDe'
    +            """.stripMargin)
    +
    +          queries.foreach(statement.execute)
    +
    +          statement.executeQuery(
    +            """
    +              |INSERT INTO TABLE addJar SELECT 'k1' as key FROM smallKV 
limit 1
    +            """.stripMargin)
    +
    +          val actualResult =
    +            statement.executeQuery("SELECT key FROM addJar")
    +          val actualResultBuffer = new 
collection.mutable.ArrayBuffer[String]()
    +          while (actualResult.next()) {
    +            actualResultBuffer += actualResult.getString(1)
    +          }
    +          actualResult.close()
    +
    +          val expectedResult =
    +            statement.executeQuery("SELECT 'k1'")
    +          val expectedResultBuffer = new 
collection.mutable.ArrayBuffer[String]()
    +          while (expectedResult.next()) {
    +            expectedResultBuffer += expectedResult.getString(1)
    +          }
    +          expectedResult.close()
    +
    +          assert(expectedResultBuffer === actualResultBuffer)
    +
    +          statement.executeQuery("DROP TABLE IF EXISTS addJar")
    +          statement.executeQuery("DROP TABLE IF EXISTS smallKV")
    +      }
    +    )
    +  }
    --- End diff --
    
    @liancheng is this test good?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to