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

    https://github.com/apache/spark/pull/21798#discussion_r203226401
  
    --- Diff: 
external/avro/src/main/scala/org/apache/spark/sql/avro/AvroFileFormat.scala ---
    @@ -276,10 +274,15 @@ private[avro] object AvroFileFormat {
         }
       }
     
    -  def ignoreFilesWithoutExtensions(conf: Configuration): Boolean = {
    -    // Files without .avro extensions are not ignored by default
    -    val defaultValue = false
    +  def ignoreExtension(conf: Configuration, options: Map[String, String]): 
Boolean = {
    +    val ignoreFilesWithoutExtensionByDefault = false
    +    val ignoreFilesWithoutExtension = conf.getBoolean(
    +      AvroFileFormat.IgnoreFilesWithoutExtensionProperty,
    +      ignoreFilesWithoutExtensionByDefault)
     
    -    conf.getBoolean(AvroFileFormat.IgnoreFilesWithoutExtensionProperty, 
defaultValue)
    +    options
    +      .get("ignoreExtension")
    --- End diff --
    
    Let's make sure we describe that in a public API later.


---

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

Reply via email to