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