From: | Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, "Drouvot, Bertrand" <bdrouvot(at)amazon(dot)com> |
Subject: | Re: Generating code for query jumbling through gen_node_support.pl |
Date: | 2023-01-26 08:39:05 |
Message-ID: | bafce03e-9536-26e4-36af-387ccc2b01ab@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 25.01.23 01:08, Michael Paquier wrote:
> On Tue, Jan 24, 2023 at 03:57:56PM +0900, Michael Paquier wrote:
>> Makes sense. That would be my intention if 0004 is the most
>> acceptable and splitting things makes things a bit easier to review.
>
> There was a silly mistake in 0004 where the jumbling code relied on
> compute_query_id rather than utility_query_id, so fixed and rebased as
> of v7 attached.
Overall, this looks good to me.
There are a couple of repetitive comments, like "typmod and collation
information are irrelevant for the query jumbling". This applies to all
nodes, so we don't need to repeat it for a number of nodes (and then not
mention it for other nodes). Maybe there should be a central place
somewhere that describes "these kinds of fields should normally be ignored".
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2023-01-26 09:07:39 | Re: improving user.c error messages |
Previous Message | Peter Eisentraut | 2023-01-26 08:37:13 | Re: Generating code for query jumbling through gen_node_support.pl |