Re: error context for vacuum to include block number - Mailing list pgsql-hackers

From Masahiko Sawada
Subject Re: error context for vacuum to include block number
Date
Msg-id CA+fd4k5O3RQp157m+p2ba61tcwtRDYE=7Ja5NCurTwE8=57ekw@mail.gmail.com
Whole thread Raw
In response to Re: error context for vacuum to include block number  (Justin Pryzby <pryzby@telsasoft.com>)
Responses Re: error context for vacuum to include block number  (Amit Kapila <amit.kapila16@gmail.com>)
List pgsql-hackers
On Fri, 27 Mar 2020 at 07:17, Justin Pryzby <pryzby@telsasoft.com> wrote:
>
> On Thu, Mar 26, 2020 at 10:04:57AM -0500, Justin Pryzby wrote:
> > Does that address your comment ?
>
> I hope so.

Thank you for updating the patch. I'm concerned a bit about overhead
of frequently updating and reverting the callback arguments in
lazy_vacuum_page(). We call that function every time when we vacuum a
page, but if the table has an index, we actually don't need to update
the callback arguments in that function. But I hope it's negligible
since all operation will be performed on memory.

Regards,

--
Masahiko Sawada            http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: potential stuck lock in SaveSlotToPath()
Next
From: Julien Rouhaud
Date:
Subject: Re: WAL usage calculation patch