Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/20387
@cloud-fan, this is a single commit on purpose because predicate push-down
makes plan changes. I think it's best to do these at once to avoid unnecessary
work. That's why I started looking more closely at push-down in the first
place: updating the other push-down code for immutable plans was a mess.
I also think it is unlikely that we will need to revert the push-down
changes here. If we end up redesigning push-down, then it is unlikely that the
easiest starting point is to roll back this fix.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
