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

    https://github.com/apache/spark/pull/23150#discussion_r238075585
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/csv/CSVSuite.scala
 ---
    @@ -1107,7 +1111,7 @@ class CSVSuite extends QueryTest with 
SharedSQLContext with SQLTestUtils with Te
     
       test("SPARK-18699 put malformed records in a `columnNameOfCorruptRecord` 
field") {
         Seq(false, true).foreach { multiLine =>
    -      val schema = new StructType().add("a", IntegerType).add("b", 
TimestampType)
    +      val schema = new StructType().add("a", IntegerType).add("b", 
DateType)
    --- End diff --
    
    I changed the type because supposed to valid date `"1983-08-04"` cannot be 
parsed with default timestamp pattern.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to