Thread: Re: Cleanup SubPlanstate

Re: Cleanup SubPlanstate

From
Andrei Lepikhov
Date:
On 10/29/24 21:34, Rafia Sabih wrote:
> Hello hackers,
> 
> While reviewing a related patch, it came to the notice that tab_eq_funcs 
> in the SubPlanState is not used. Hence the patch.
Ok.
My initial doubt in such cases is - may it be the case where we mask a 
mistake? But that case looks good. Introduced in ab05eed, the approach 
with an equality function for each grouping column was replaced by 
bf6c614, which moved the grouping operation execution via the expression 
evaluation machinery. That made the tab_eq_funcs useless.
So, I think it may be removed.

-- 
regards, Andrei Lepikhov