Re: Stability of queryid in minor versions

From: David Rowley <dgrowleyml(at)gmail(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Stability of queryid in minor versions
Date: 2024-04-15 02:54:52
Message-ID: CAApHDvor4FE6ZuuatLEgvGztHZ_azqiSrhsHahH2hH7tcw8w5A@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, 15 Apr 2024 at 14:09, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>
> On Mon, Apr 15, 2024 at 01:31:47PM +1200, David Rowley wrote:
> > I'm unsure if "Rule of thumb" is the correct way to convey that. We
> > can't really write "We endeavour to", as who is "We". Maybe something
> > like "Generally, it can be assumed that queryid is stable between all
> > minor versions of a major version of ..., providing that <other
> > reasons>".
>
> It sounds to me that the term "best-effort" is adapted here? Like in
> "The compatibility of query IDs is preserved across minor versions on
> a best-effort basis. It is possible that the post-parse-analysis tree
> changes across minor releases, impacting the value of queryid for the
> same query run across two different minor versions.".

I had another try and ended up pushing the logical / physical replica
details up to the paragraph above. It seems more relevant to mention
this in the section which details reasons why the queryid can be
unstable due to metadata variations. I think keeping the 2nd
paragraph for reasons it's stable is a good separation of
responsibility. I didn't include the "best-effort" word, but here's
what I did come up with.

David

Attachment Content-Type Size
doc_clarify_queryid_stability_v2.patch text/plain 2.2 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message David Steele 2024-04-15 03:17:51 Re: post-freeze damage control
Previous Message David G. Johnston 2024-04-15 02:23:33 Re: Stability of queryid in minor versions