You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2020/10/21 14:58:44 UTC

[GitHub] [spark] cloud-fan commented on pull request #29831: [SPARK-32351][SQL] Show partially pushed down partition filters in explain()

cloud-fan commented on pull request #29831:
URL: https://github.com/apache/spark/pull/29831#issuecomment-713639134


   > The current code assumes the same partition filters are generated again in FileSourceStrategy and passed into FileSourceScanExec.
   
   A further refactor can be: remove the partition filters after they are pushed down into the catalog file index in the optimizer rule. Also keep the partition filters in the query plan so that we can explain them.


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



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