Re: Ignore 2PC transaction GIDs in query jumbling

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Julien Rouhaud <rjuju123(at)gmail(dot)com>
Cc: Postgres hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Ignore 2PC transaction GIDs in query jumbling
Date: 2023-08-13 06:25:33
Message-ID: ZNh3XV2+cubcVaWk@paquier.xyz
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Aug 01, 2023 at 10:46:58AM +0800, Julien Rouhaud wrote:
> On Tue, Aug 01, 2023 at 11:37:49AM +0900, Michael Paquier wrote:
>> On Tue, Aug 01, 2023 at 10:22:09AM +0800, Julien Rouhaud wrote:
>>> Looking at the rest of the ignored patterns, the only remaining one would be
>>> DEALLOCATE, which AFAICS doesn't have a query_jumble_ignore tag for now.
>>
>> This one seems to be simple as well with a location field, looking
>> quickly at its Node.
>
> Agreed, it should be as trivial to implement as for the 2pc commands :)

Perhaps not as much, actually, because I was just reminded that
DEALLOCATE is something that pg_stat_statements ignores. So this
makes harder the introduction of tests. Anyway, I guess that your own
extension modules have a need for a query ID compiled with these
fields ignored?

For now, I have applied the 2PC bits independently, as of 638d42a.
--
Michael

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Julien Rouhaud 2023-08-13 06:48:22 Re: Ignore 2PC transaction GIDs in query jumbling
Previous Message Noah Misch 2023-08-13 03:15:31 pg_waldump vs. all-zeros WAL files; server creation of such files