Horton

Re: error context for vacuum to include block number

Andres Freund <andres@anarazel.de>

View thread View on postgr.es
From: Andres Freund <andres@anarazel.de>
To: Justin Pryzby <pryzby@telsasoft.com>
Cc: Alvaro Herrera <alvherre@2ndquadrant.com>, Michael Paquier <michael@paquier.xyz>, pgsql-hackers@postgresql.org
Date: 2020-01-20T19:13:05Z
Lists: pgsql-hackers 
Hi,

On 2019-12-12 21:08:31 -0600, Justin Pryzby wrote:
> On Wed, Dec 11, 2019 at 12:33:53PM -0300, Alvaro Herrera wrote:
> On Wed, Dec 11, 2019 at 08:54:25AM -0800, Andres Freund wrote:
> > Hm, wonder if could be worthwhile to not use a separate struct here, but
> > instead extend one of the existing structs to contain the necessary
> > information. Or perhaps have one new struct with all the necessary
> > information. There's already quite a few places that do
> > get_namespace_name(), for example.
> 
> Didn't find a better struct to use yet.

I was thinking that you could just use LVRelStats.

Greetings,

Andres Freund

Commits

  1. Avoid calls to RelationGetRelationName() and RelationGetNamespace() in

  2. Introduce vacuum errcontext to display additional information.

  3. Fix mesurement of elapsed time during truncating heap in VACUUM.

  4. Allow vacuum command to process indexes in parallel.

  5. Refactor code dedicated to index vacuuming in vacuumlazy.c

  6. Remove duplicated progress reporting during heap scan of VACUUM