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

    https://github.com/apache/spark/pull/21773#discussion_r202550099
  
    --- Diff: 
external/avro/src/test/scala/org/apache/spark/sql/avro/AvroSuite.scala ---
    @@ -37,27 +37,27 @@ import org.apache.spark.sql.test.{SharedSQLContext, 
SQLTestUtils}
     import org.apache.spark.sql.types._
     
     class AvroSuite extends QueryTest with SharedSQLContext with SQLTestUtils {
    -  val episodesFile = "src/test/resources/episodes.avro"
    -  val testFile = "src/test/resources/test.avro"
    +  val episodesAvro = "episodes.avro"
    +  val testAvro = "test.avro"
    --- End diff --
    
    Nit: 
    Maybe we can make it simpler
    ```
    val episodesAvro = testFile("episodes.avro")
    val testAvro = testFile("test.avro")
    ```



---

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

Reply via email to