Re: relfilenode statistics

Hi,

On Mon, Aug 05, 2024 at 05:28:22AM +0000, Bertrand Drouvot wrote:
> Hi,
>
> On Thu, Jul 11, 2024 at 06:10:23AM +0000, Bertrand Drouvot wrote:
> > Hi,
> >
> > On Thu, Jul 11, 2024 at 01:58:19PM +0900, Michael Paquier wrote:
> > > On Wed, Jul 10, 2024 at 01:38:06PM +0000, Bertrand Drouvot wrote:
> > > > So, I think it makes sense to link the hashkey to all the RelFileLocator
> > > > fields, means:
> > > >
> > > > dboid (linked to RelFileLocator's dbOid)
> > > > objoid (linked to RelFileLocator's spcOid)
> > > > relfile (linked to RelFileLocator's relNumber)
> > >
> > > Hmm. How about using the table OID as objoid,
> >
> > The issue is that we don't have the relation OID when writing buffers out (that's
> > one of the reason explained in [1]).
> >
> > [1]: https://www.postgresql.org/message-id/Zl2k8u4HDTUW6QlC%40ip-10-97-1-34.eu-west-3.compute.internal
> >
> > Regards,
> >
>
> Please find attached a mandatory rebase due to the recent changes around
> statistics.
>
> As mentioned up-thread:
>
> The attached patch is not in a fully "polished" state yet: there is more places
> we should add relfilenode counters, create more APIS to retrieve the relfilenode
> stats....
>
> It is in a state that can be used to discuss the approach it is implementing (as
> we have done so far) before moving forward.

Please find attached a mandatory rebase.

In passing, checking if based on the previous discussion (and given that we
don't have the relation OID when writing buffers out) you see another approach
that the one this patch is implementing?

Regards,

--
Bertrand Drouvot
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com