add `PhysicalExpr` to support, Selective Aggregates feature using `FILTER` clause
Which issue does this PR close?
#2214
Add support for adding filter to aggregation.
eg: AGGREGATE(input) FILTER (WHERE condition)
Closes #2214
What changes are included in this PR?
ExprWithFilteris added so that expr can be evaluated based on the given filter.ExprWithFilteris used in aggregate functions if it contains filter- filtered column name will be added to the TableScan's
projectionby making expression visitors to visit aggregate function filters
@andygrove it's done. Please have a look and let me know :)
I am not sure what the current status of this PR is and it is not passing CI, so marking it as "draft" -- please mark it as ready for review when it is
@alamb I've fixed the test. Yes, it's ready for review.
@alamb PTAL when you can. LGTM but I have not been working so closely with physical execution lately.
I am sorry I don't have time to review this PR 😭
Hey @alamb
No worries, I’ll close this for now. We can rework this when this is relevant again or someone can take this work again and patch it
Thanks and sorry @poonai