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

    https://github.com/apache/spark/pull/21769#discussion_r202524884
  
    --- Diff: 
external/avro/src/main/scala/org/apache/spark/sql/avro/AvroFileFormat.scala ---
    @@ -64,7 +64,7 @@ private[avro] class AvroFileFormat extends FileFormat 
with DataSourceRegister {
         // Schema evolution is not supported yet. Here we only pick a single 
random sample file to
         // figure out the schema of the whole dataset.
         val sampleFile =
    -      if 
(conf.getBoolean(AvroFileFormat.IgnoreFilesWithoutExtensionProperty, true)) {
    +      if (AvroFileFormat.ignoreFilesWithoutExtensions(conf)) {
    --- End diff --
    
    Here is how people use the option so far: 
https://github.com/databricks/spark-avro/issues/71#issuecomment-280844562 . 
Probably we should seperatly from the PR discuss how we are going to fix the 
"bug" and break backward compatibily.


---

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

Reply via email to