Github user gatorsmile commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22654#discussion_r224626882
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
 ---
    @@ -1826,4 +1826,13 @@ class CSVSuite extends QueryTest with 
SharedSQLContext with SQLTestUtils with Te
         val df = spark.read.option("enforceSchema", false).csv(input)
         checkAnswer(df, Row("1", "2"))
       }
    +
    +  test("using the backward slash as the delimiter") {
    +    val input = Seq("""abc\1""").toDS()
    --- End diff --
    
    If a user specified `\` as a delimiter, we should issue an exception 
message and let users add the escape symbol i.e., `\\`. It is weird to see both 
`\\` and `\` are representing the same thing `\`. We should be consistent for 
handling backslash in all the cases.  


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to