On Sat, 10 Jul 2021 at 03:15, Zhihong Yu <zyu@yugabyte.com> wrote:
> I was looking at find_hash_columns() in nodeAgg.c
>
> It seems the first loop tries to determine the max column number needed, along with whether all columns are needed.
>
> The loop can be re-written as shown in the patch.
This runs during ExecInitAgg(). Do you have a test case where you're
seeing any performance gains from this change?
David