Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/19470#discussion_r144472506
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -2050,4 +2050,64 @@ class SQLQuerySuite extends QueryTest with
SQLTestUtils with TestHiveSingleton {
}
}
}
+
+ Seq("orc", "parquet").foreach { format =>
+ test(s"SPARK-18355 Read data from a hive table with a new column -
$format") {
+ val client =
spark.sharedState.externalCatalog.asInstanceOf[HiveExternalCatalog].client
+
+ Seq("true", "false").foreach { value =>
+ withSQLConf(
+ HiveUtils.CONVERT_METASTORE_ORC.key -> value,
+ HiveUtils.CONVERT_METASTORE_PARQUET.key -> value) {
--- End diff --
Thank you for review, @viirya . For that, yes, we can, but that will be a
little-bit overkill.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
