Re: pgstattuple: Have pgstattuple_approx accept TOAST tables - Mailing list pgsql-hackers

From Laurenz Albe
Subject Re: pgstattuple: Have pgstattuple_approx accept TOAST tables
Date
Msg-id e8850a0f4cb0609c210a4905141255e0622ef934.camel@cybertec.at
Whole thread Raw
In response to pgstattuple: Have pgstattuple_approx accept TOAST tables  (Peter Eisentraut <peter.eisentraut@2ndquadrant.com>)
Responses Re: pgstattuple: Have pgstattuple_approx accept TOAST tables  (Peter Eisentraut <peter.eisentraut@2ndquadrant.com>)
List pgsql-hackers
On Fri, 2020-04-17 at 13:01 +0200, Peter Eisentraut wrote:
> I alluded to this in [0], but it's better discussed in its own thread.
> 
> I think the check that makes pgstattuple_approx reject TOAST tables is a 
> mistake.  They have visibility and free space map, and it works just 
> fine if the check is removed.
> 
> Attached is a patch to fix this and add some tests related to how 
> pgstattuple and pg_visibility accept TOAST tables for inspection.
> 
> 
> [0]: 
> https://www.postgresql.org/message-id/dc35a398-37d0-75ce-07ea-1dd71d98f8ec@2ndquadrant.com

I gave the patch a spin, and it passes regression tests and didn't
cause any problems when I played with it.

No upgrade or dump considerations, of course.

This is a clear improvement.

I'll mark the patch as "ready for committer".

Yours,
Laurenz Albe




pgsql-hackers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: Resetting spilled txn statistics in pg_stat_replication
Next
From: Fujii Masao
Date:
Subject: Re: min_safe_lsn column in pg_replication_slots view