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

    https://github.com/apache/spark/pull/21288#discussion_r187764083
  
    --- Diff: 
sql/core/src/test/scala/org/apache/spark/sql/FilterPushdownBenchmark.scala ---
    @@ -32,14 +32,14 @@ import org.apache.spark.util.{Benchmark, Utils}
      */
     object FilterPushdownBenchmark {
       val conf = new SparkConf()
    -  conf.set("orc.compression", "snappy")
    -  conf.set("spark.sql.parquet.compression.codec", "snappy")
    +    .setMaster("local[1]")
    +    .setAppName("FilterPushdownBenchmark")
    +    .set("spark.driver.memory", "3g")
    --- End diff --
    
    these and master - change to setIfMissing()? I think it's great if these 
can be set via config


---

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

Reply via email to