From: | Andreas Karlsson <andreas(at)proxel(dot)se> |
---|---|
To: | Abdoulaye Ba <abdoulayeba29(at)gmail(dot)com> |
Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: PATCH: Add hooks for pg_total_relation_size and pg_indexes_size |
Date: | 2024-08-28 15:53:36 |
Message-ID: | d9d7b9ce-67a3-44ec-b191-ecd736ad82e6@proxel.se |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 8/9/24 6:59 PM, Abdoulaye Ba wrote:> The primary use case for
this hook is to allow extensions to account
> for additional storage mechanisms that are not covered by the
> default PostgreSQL relation size calculations. For instance, in our
> project, we are working with an external indexing system (Tantivy)
> that maintains additional data structures outside the standard
> PostgreSQL storage. This hook allows us to include the size of these
> additional structures in the total relation size calculations.
>
> While I understand your suggestion about custom index AMs, the
> intent behind this hook is broader. It is not limited to custom
> index types but can also be used for other forms of external storage
> that need to be accounted for in relation size calculations. This is
> why a generic callback hook was chosen over extending the index AM
> interface.
>
> However, if there is a consensus that such a hook would be better
> suited within the index AM interface for cases involving custom
> index storage, I'm open to discussing this further and exploring how
> it could be integrated more tightly with the existing PostgreSQL AM
> framework.
Yeah, I strongly suspected it was ParadeDB. :)
I am only one developer but I really do not like solving this with a
hook, instead I think the proper solution is to integrate this properly
with custom AMs and storage managers. I think we should do it properly
or not at all.
Andreas
From | Date | Subject | |
---|---|---|---|
Next Message | Matthias van de Meent | 2024-08-28 15:59:32 | Re: Reading all tuples in Index Access Method |
Previous Message | Andreas Karlsson | 2024-08-28 15:26:38 | Re: Little cleanup of ShmemInit function names |