Re: Convert planner's AggInfo and AggTransInfo to Nodes

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Dagfinn Ilmari Mannsåker <ilmari(at)ilmari(dot)org>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: Convert planner's AggInfo and AggTransInfo to Nodes
Date: 2022-07-19 16:31:08
Message-ID: 1210832.1658248268@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

=?utf-8?Q?Dagfinn_Ilmari_Manns=C3=A5ker?= <ilmari(at)ilmari(dot)org> writes:
> It seems like a reasonable idea, but I don't know enough to judge the
> wider ramifications of it. But one thing that the patch should also do,
> is switch to using the l*_node() functions instead of manual casting.

Hm, I didn't bother with that on the grounds that there's no question
what should be in those two lists. But I guess it's not much extra
code, so pushed that way.

regards, tom lane

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2022-07-19 16:36:31 Re: Memory leak fix in psql
Previous Message Jacob Champion 2022-07-19 16:30:18 Re: [PATCH] Log details for client certificate failures