cloud-fan commented on a change in pull request #24068: [SPARK-27105][SQL] 
Optimize away exponential complexity in ORC predicate conversion
URL: https://github.com/apache/spark/pull/24068#discussion_r288938467
 
 

 ##########
 File path: 
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/FilterPushdownBenchmark.scala
 ##########
 @@ -135,6 +139,34 @@ object FilterPushdownBenchmark extends BenchmarkBase with 
SQLHelper {
     benchmark.run()
   }
 
+  def filterPushDownBenchmarkWithColumn(
 
 Review comment:
   a general note: I think `FilterPushdownBenchmark` is to test how good are 
these file formats about skipping disk IO with pushed filter. Checking the 
performance of building the filters may go beyond its scope.
   
   Maybe we can just do a manual microbenchmark, and post the benchmark and 
result in PR description.
   
   cc @dongjoon-hyun 

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

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

Reply via email to