public inbox for [email protected]
help / color / mirror / Atom feedFrom: Michael Paquier <[email protected]>
To: Julien Rouhaud <[email protected]>
Cc: Sami Imseih <[email protected]>
Cc: Lukas Fittl <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Cc: Marko M <[email protected]>
Cc: Alvaro Herrera <[email protected]>
Subject: Re: [PATCH] Optionally record Plan IDs to track plan changes for a query
Date: Wed, 12 Feb 2025 10:59:04 +0900
Message-ID: <[email protected]> (raw)
In-Reply-To: <Z6v3dSJ7yEJ9xai4@jrouhaud>
References: <[email protected]>
<CAA5RZ0vM9AsEqvKued2drKZJ1opt3wbYaDbxGzi-khkNzwn7og@mail.gmail.com>
<[email protected]>
<CAA5RZ0sfRbd1xcq_oHNA0TPr57yM8qkg-GoD6A0nUfyxZhs33Q@mail.gmail.com>
<CAP53PkxocbNr+eRag3FEJp3-7S1U80FspOg8UQjO902TWMG=6A@mail.gmail.com>
<CAA5RZ0u6yJdFL=p5vdpbZFS-2YY+Z6vtzmt4gejgZa3RcNiWMQ@mail.gmail.com>
<[email protected]>
<CAA5RZ0t80hP2aTv97QtEJy39GkxKmDBVDiTBApfiuTa4O=TEWQ@mail.gmail.com>
<[email protected]>
<Z6v3dSJ7yEJ9xai4@jrouhaud>
On Wed, Feb 12, 2025 at 09:20:53AM +0800, Julien Rouhaud wrote:
> On Wed, Feb 12, 2025 at 09:08:00AM +0900, Michael Paquier wrote:
>> Wikipedia seems to agree with you that "fingerprint" would fit for
>> this purpose, though:
>> https://en.wikipedia.org/wiki/Fingerprint_(computing)
>>
>> Has anybody any comments about that? That would be a large renaming,
>> but in the long term is makes sense if we want to apply that to more
>> than just parse nodes and query strings. If you do that, it impacts
>> the file names and the properties, that are hidden in the backend for
>> most of it, except the entry API and JumbleState. This last part
>> impacts some extensions and I have been maintaining one a bit
>> (pg_hint_plan).
>
> I agree that fingerprint is a good improvement.
Okay, thanks. So this would mean something for the file names, the
node_attr names, the structures and the APIs if we put all that under
the same label.
> >
> > Also adding Julien in CC,
> > as he has some out-of-core extension code that depends on the jumbling
> > structures if I recall correctly.
>
> I do have an extension to support custom fingerprinting logic, but the
> introduction of the pg_node_attr based jumbling kind of broke it.
>
> FTR my main motivation was to be able to deal with queries referencing
> temporary relations, as if your application creates a lot of those it basically
> means that you cannot use pg_stat_statements anymore.
Do you have an issue more details about your problem? If we can
improve the situation in core without impacting the existing cases
that we need to support in pgss, that may be worth looking at.
--
Michael
Attachments:
[application/pgp-signature] signature.asc (833B, 2-signature.asc)
download
view thread (34+ messages) latest in thread
reply
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: [email protected]
Cc: [email protected], [email protected], [email protected], [email protected], [email protected], [email protected], [email protected]
Subject: Re: [PATCH] Optionally record Plan IDs to track plan changes for a query
In-Reply-To: <[email protected]>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox