Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21259#discussion_r186503298
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveStrategies.scala ---
@@ -189,10 +189,12 @@ case class RelationConversions(
// Return true for Apache ORC and Hive ORC-related configuration names.
// Note that Spark doesn't support configurations like
`hive.merge.orcfile.stripe.level`.
private def isOrcProperty(key: String) =
- key.startsWith("orc.") || key.contains(".orc.")
+ conf.getConf(HiveUtils.CONVERT_METASTORE_TABLE_PROPERTY) &&
+ (key.startsWith("orc.") || key.contains(".orc."))
private def isParquetProperty(key: String) =
- key.startsWith("parquet.") || key.contains(".parquet.")
+ conf.getConf(HiveUtils.CONVERT_METASTORE_TABLE_PROPERTY) &&
+ key.startsWith("parquet.") || key.contains(".parquet.")
--- End diff --
is this missing `(` and `)`?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
