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

2023-07-13 Thread via GitHub
gdhuper commented on code in PR #41904: URL: https://github.com/apache/spark/pull/41904#discussion_r1262174669 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/CSVOptions.scala: ## @@ -253,16 +253,18 @@ class CSVOptions( /** * A string between two consecut

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

2023-07-12 Thread via GitHub
gdhuper commented on code in PR #41904: URL: https://github.com/apache/spark/pull/41904#discussion_r1261808401 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/CSVOptions.scala: ## @@ -253,16 +253,18 @@ class CSVOptions( /** * A string between two consecut

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

2023-07-12 Thread via GitHub
gdhuper commented on code in PR #41904: URL: https://github.com/apache/spark/pull/41904#discussion_r1261777186 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/CSVOptions.scala: ## @@ -253,16 +253,18 @@ class CSVOptions( /** * A string between two consecut

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

2023-07-11 Thread via GitHub
gdhuper commented on code in PR #41904: URL: https://github.com/apache/spark/pull/41904#discussion_r1259340808 ## sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/csv/CSVOptions.scala: ## @@ -253,16 +253,18 @@ class CSVOptions( /** * A string between two consecut