Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/19199#discussion_r138271989
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVFileFormat.scala
---
@@ -109,6 +109,20 @@ class CSVFileFormat extends TextBasedFileFormat with
DataSourceRegister {
}
}
+ if (requiredSchema.length == 1 &&
+ requiredSchema.head.name == parsedOptions.columnNameOfCorruptRecord)
{
+ throw new AnalysisException(
+ "Since Spark 2.3, the queries from raw JSON/CSV files are
disallowed when the\n" +
+ "referenced columns only include the internal corrupt record
column\n" +
+ s"(named ${parsedOptions.columnNameOfCorruptRecord} by default).
For example:\n" +
--- End diff --
Here `named ${parsedOptions.columnNameOfCorruptRecord}` looks weird. As it
is interpolated to configured column name in runtime. We should replace it as:
1. "named `_corrupt_record` by default", as same as in migration guide.
2. "named by the config `spark.sql.columnNameOfCorruptRecord`".
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
