gengliangwang commented on pull request #28741: URL: https://github.com/apache/spark/pull/28741#issuecomment-640172081
@wangyum Thanks for point it out. I made a mistake. This solution is not as powerful as CNF conversion since it can't even push down such predicate ``` (p1(t1) or p2(t2)) and p3(t1) ``` to `t1` with CNF we can push down ``` p1(t1) or p3(t1) ``` I am closing this one and reopening #28733 ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
