Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19719#discussion_r150362312
  
    --- Diff: 
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2Suites.scala
 ---
    @@ -521,7 +521,20 @@ class HiveThriftBinaryServerSuite extends 
HiveThriftJdbcTest {
             conf += resultSet.getString(1) -> resultSet.getString(2)
           }
     
    -      assert(conf.get("spark.sql.hive.metastore.version") === 
Some("1.2.1"))
    +      assert(conf.get("spark.sql.hive.version") === Some("1.2.1"))
    +    }
    +  }
    +
    +  test("Checks Hive version via SET") {
    +    withJdbcStatement() { statement =>
    +      val resultSet = statement.executeQuery("SET")
    +
    +      val conf = mutable.Map.empty[String, String]
    +      while (resultSet.next()) {
    +        conf += resultSet.getString(1) -> resultSet.getString(2)
    +      }
    +
    +      assert(conf.get("spark.sql.hive.version") === Some("1.2.1"))
    --- End diff --
    
    a default value will be returned, isn't it?


---

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

Reply via email to