maropu edited a comment on pull request #29270:
URL: https://github.com/apache/spark/pull/29270#issuecomment-667439259


   > One of my worries is that: this test generates plans with empty tables, 
and we lost test coverage for things like SMJ. Can the variant feature help to 
improve the test coverage here?
   
   Interesting point! How about injecting stats data when creating TPCDS 
tables? https://github.com/apache/spark/compare/master...maropu:SPARK-32466 In 
some quries, this might make it possible to make generated plans closer to the 
actual ones.


----------------------------------------------------------------
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:
us...@infra.apache.org



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

Reply via email to