Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22410#discussion_r217672194
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -806,6 +807,8 @@ private[spark] class HiveExternalCatalog(conf:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22410#discussion_r217669057
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -806,6 +807,8 @@ private[spark] class HiveExternalCatalog(con
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22410#discussion_r217623551
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -1309,6 +1312,8 @@ object HiveExternalCatalog {
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22410
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22410#discussion_r217603253
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -1309,6 +1312,8 @@ object HiveExternalCatalog {
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22410#discussion_r217592782
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -1309,6 +1312,8 @@ object HiveExternalCatalog {
val
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22410#discussion_r217590735
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -1309,6 +1312,8 @@ object HiveExternalCatalog {
GitHub user ueshin opened a pull request:
https://github.com/apache/spark/pull/22410
[SPARK-25418][SQL] The metadata of DataSource table should not include
Hive-generated storage properties.
## What changes were proposed in this pull request?
When Hive support enabled, Hive