Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22814#discussion_r228063256
  
    --- Diff: 
external/avro/src/main/scala/org/apache/spark/sql/avro/AvroOptions.scala ---
    @@ -79,4 +80,23 @@ class AvroOptions(
       val compression: String = {
         
parameters.get("compression").getOrElse(SQLConf.get.avroCompressionCodec)
       }
    +
    +  @transient private val acceptableParseMode = Seq(PermissiveMode, 
FailFastMode)
    --- End diff --
    
    do we need to check mode here? The `AvroOptions` can be used with reading 
avro files, and DROP MALFORMED can be supported.


---

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

Reply via email to