Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20894#discussion_r191824714
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/csv/CSVDataSource.scala
---
@@ -110,14 +114,81 @@ abstract class CSVDataSource extends Serializable {
}
}
-object CSVDataSource {
+object CSVDataSource extends Logging {
def apply(options: CSVOptions): CSVDataSource = {
if (options.multiLine) {
MultiLineCSVDataSource
} else {
TextInputCSVDataSource
}
}
+
+ /**
+ * Checks that column names in a CSV header and field names in the
schema are the same
+ * by taking into account case sensitivity.
+ */
--- End diff --
To help readability, how about at least adding `@param columnNames` and
`@param schema`?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
