Re: Cleanup: remove unused fields from nodes - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Cleanup: remove unused fields from nodes
Date
Msg-id 3596755.1713927820@sss.pgh.pa.us
Whole thread Raw
In response to Re: Cleanup: remove unused fields from nodes  (Michael Paquier <michael@paquier.xyz>)
Responses Re: Cleanup: remove unused fields from nodes
List pgsql-hackers
Michael Paquier <michael@paquier.xyz> writes:
> On Tue, Apr 23, 2024 at 01:01:04PM -0400, Tom Lane wrote:
>> That is, query jumbling no longer distinguishes "DEALLOCATE x" from
>> "DEALLOCATE ALL", because the DeallocateStmt.name field is marked
>> query_jumble_ignore.  Now maybe that's fine, but it's a point
>> we'd not considered so far in this thread.  Thoughts?

> And of course, I've managed to forget about bb45156f342c and the
> reason behind the addition of the field is to be able to make the
> difference between the named and ALL cases for DEALLOCATE, around
> here:
> https://www.postgresql.org/message-id/ZNq9kRwWbKzvR%2B2a%40paquier.xyz

Hah.  Seems like the comment for isall needs to explain that it
exists for this purpose, so we don't make this mistake again.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: Cleanup: remove unused fields from nodes
Next
From: Tatsuo Ishii
Date:
Subject: Re: Row pattern recognition