[GitHub] [spark] srowen commented on a diff in pull request #41904: [SPARK-43389][SQL] Added a null check for lineSep option

srowen commented on code in PR #41904: URL: https://github.com/apache/spark/pull/41904#discussion_r1261808987 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/CSVOptions.scala: ## @@ -253,16 +253,18 @@ class CSVOptions( /** * A string between two consecuti

[GitHub] [spark] srowen commented on a diff in pull request #41904: [SPARK-43389][SQL] Added a null check for lineSep option

srowen commented on code in PR #41904: URL: https://github.com/apache/spark/pull/41904#discussion_r1261781671 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/CSVOptions.scala: ## @@ -253,16 +253,18 @@ class CSVOptions( /** * A string between two consecuti

[GitHub] [spark] srowen commented on a diff in pull request #41904: [SPARK-43389][SQL] Added a null check for lineSep option

srowen commented on code in PR #41904: URL: https://github.com/apache/spark/pull/41904#discussion_r1259650448 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/CSVOptions.scala: ## @@ -253,16 +253,18 @@ class CSVOptions( /** * A string between two consecuti