Greetings,
* Thomas Munro (thomas.munro@gmail.com) wrote:
> On Fri, Dec 11, 2020 at 7:57 AM Stephen Frost <sfrost@snowman.net> wrote:
> > * Tom Lane (tgl@sss.pgh.pa.us) wrote:
> > > The if-we're-going-to-delay-anyway path in vacuum_delay_point seems
> > > OK to add a touch more overhead to, though.
> >
> > Alright, for this part at least, seems like it'd be something like the
> > attached.
> >
> > Only lightly tested, but does seem to address the specific example which
> > was brought up on this thread.
> >
> > Thoughts..?
>
> +1
Thanks for that. Attached is just a rebased version with a commit
message added. If there aren't any other concerns, I'll commit this in
the next few days and back-patch it. When it comes to 12 and older,
does anyone want to opine about the wait event to use? I was thinking
PG_WAIT_TIMEOUT or WAIT_EVENT_PG_SLEEP ...
Or do folks think this shouldn't be backpatched? That would mean it
wouldn't help anyone for years, which would be pretty unfortuante, hence
my feeling that it's worthwhile to backpatch.
Thanks!
Stephen