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

    https://github.com/apache/spark/pull/21122#discussion_r186243450
  
    --- Diff: 
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/HiveDDLSuite.scala 
---
    @@ -1354,7 +1354,8 @@ class HiveDDLSuite
         val indexName = tabName + "_index"
         withTable(tabName) {
           // Spark SQL does not support creating index. Thus, we have to use 
Hive client.
    -      val client = 
spark.sharedState.externalCatalog.asInstanceOf[HiveExternalCatalog].client
    +      val client =
    +        
spark.sharedState.externalCatalog.unwrapped.asInstanceOf[HiveExternalCatalog].client
    --- End diff --
    
    We plan to get rid of `HiveSessionStateBuilder` and treat `Hive` tables as 
the regular data source tables. `Hive` should be the same as the other external 
data sources. For the other data sources, we will not have a shared state and 
session state. Hive metastore is just used for global metastore. In the future, 
it is also pluggable. Thus, it does not make sense to re-introduce Hive 
SharedState in the current stage. 


---

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

Reply via email to