Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17631#discussion_r147544807
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JdbcUtils.scala
---
@@ -119,7 +119,7 @@ object JdbcUtils extends Logging {
isCaseSensitive: Boolean,
dialect: JdbcDialect): String = {
val columns = if (tableSchema.isEmpty) {
- rddSchema.fields.map(x =>
dialect.quoteIdentifier(x.name)).mkString(",")
+ rddSchema.fields.map(x =>
dialect.quoteIdentifier(x.name.replace("\"", ""))).mkString(",")
--- End diff --
This will replace all the quotes. We just need to `stripPrefix` and
`stripSuffix` of double quotes.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
