public inbox for [email protected]
help / color / mirror / Atom feedFrom: Andres Freund <[email protected]>
To: Peter Geoghegan <[email protected]>
Cc: PostgreSQL Hackers <[email protected]>
Subject: Re: Nonrandom scanned_pages distorts pg_class.reltuples set by VACUUM
Date: Thu, 17 Feb 2022 16:18:22 -0800
Message-ID: <[email protected]> (raw)
In-Reply-To: <CAH2-Wzk+oeG1S_OxJH67ncQ8QQ8EDGmfAeZzwRUR=_A_pt4SDg@mail.gmail.com>
References: <CAH2-WzkNKORurux459M64mR63Aw4Jq7MBRVcX=CvALqN3A88WA@mail.gmail.com>
<CAH2-WzkNDbL9dKTwq+CAzkMk-XO=duYSZ-iLLjE=FpMujpahzQ@mail.gmail.com>
<[email protected]>
<CAH2-Wz=uuc5W8aa9hWcttE2QMrkJT8MSEATeQ34HvJYjAdBY7g@mail.gmail.com>
<[email protected]>
<CAH2-Wzk+oeG1S_OxJH67ncQ8QQ8EDGmfAeZzwRUR=_A_pt4SDg@mail.gmail.com>
Hi,
On 2022-02-17 14:23:51 -0800, Peter Geoghegan wrote:
> On Thu, Feb 17, 2022 at 10:33 AM Andres Freund <[email protected]> wrote:
> > On 2022-02-16 19:43:09 -0800, Peter Geoghegan wrote:
> > > It makes the code in vacuumlazy.c much cleaner. In fact, that's how commit
> > > 44fa8488 started off -- purely as refactoring work.
> >
> > The problem is that it didn't end up as that. You combined refactoring with
> > substantial changes. And described it large and generic terms.
>
> What substantial changes are you referring to? The one thing that did
> change was the commit message, which framed everything in terms of the
> later work. It really is true that the patch that I committed was
> essentially the same patch as the one posted on November 22, in both
> style and substance. Before I really even began to think about the
> freezing stuff. This is a matter of record.
Here I was referencing your description of how the patch started ("purely as
refactoring work"), and then evolved into something not just a refactoring.
> > It's a contentious thread. I asked for things to be split. In that context,
> > it's imo common curtesy for non-trivial patches to write something like
>
> I didn't see a way to usefully split up 0001 any further (having
> already split it up into 0001 and 0002). That's not particularly
> obvious, but it's true.
If helpful I can give a go at showing how I think it could be split up. Or
perhaps more productively, do that on a not-yet-committed larger patch.
Greetings,
Andres Freund
view thread (4+ 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]
Subject: Re: Nonrandom scanned_pages distorts pg_class.reltuples set by VACUUM
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