Re: pgsql: Don't consider newly inserted tuples in nbtree VACUUM.

From: Peter Geoghegan <pg(at)bowt(dot)ie>
To: Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>
Cc: Michael Paquier <michael(at)paquier(dot)xyz>, pgsql-committers <pgsql-committers(at)lists(dot)postgresql(dot)org>
Subject: Re: pgsql: Don't consider newly inserted tuples in nbtree VACUUM.
Date: 2021-03-11 20:55:24
Message-ID: CAH2-Wz=TVSLTJLxtJ+od5vFn8abNwsc3b__oNDHBX8fZ3+FPaQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On Thu, Mar 11, 2021 at 12:39 PM Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> wrote:
> Given the short life of recheck_on_update I don't think we should
> consider it a precedent.

Now we have a useful precedent. Apparently there is no way to truly
remove a storage parameter. That seems a little annoying, but not
worth spending too much time on.

> > I now accept that the dump/reload hazards when upgrading to 14 are not
> > acceptable. ISTM that adding back the reloption on master/Postgres 14
> > (without doing anything with it) is the only way to fix everything.
>
> Yeah, I agree. It would be slightly better to ignore it on input (ie.
> make ALTER TABLE SET a no-op for that case), but I don't think we have
> code to do that.

That would be better, certainly.

> > Any thoughts on that plan? I can do that in the next few hours if
> > there are no objections.
>
> Sounds good to me.

I just pushed a fix that does that. In addition, I reverted the
changes to REL_11_STABLE and REL_12_STABLE, meaning that crake will go
back to testing

Thanks
--
Peter Geoghegan

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Geoghegan 2021-03-11 20:55:26 pgsql: Restore vacuum_cleanup_index_scale_factor coverage.
Previous Message Peter Geoghegan 2021-03-11 20:43:35 pgsql: Add back vacuum_cleanup_index_scale_factor parameter.