[GitHub] [spark] viirya commented on pull request #35669: [SPARK-38041][SQL]DataFilter pushed down with PartitionFilter

[GitBox <gi...@apache.org>]

viirya commented on pull request #35669:
URL: https://github.com/apache/spark/pull/35669#issuecomment-1054549102


   Hmm, the first glance looks a bit confused, actually. Maybe it is good to improve the PR description. So, I guess this means, if the data filter is partition related, e.g. a < 10 is only for partition c = 0, and a >= 10 is for partitions c >= 1 and c < 3, then once we strip out partition filter during pushing down, we will end with a < 0 or a >= 10 which actually scans all data.
   
   But we only want a < 10 in partition c = 0, and a >= 10 in other two partitions.
   
   Is it correct?
   
   
   


-- 
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.

To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org

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