From: | Jeff Davis <pgsql(at)j-davis(dot)com> |
---|---|
To: | Simon Riggs <simon(at)2ndquadrant(dot)com> |
Cc: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Reloptions for table access methods |
Date: | 2020-12-15 23:59:02 |
Message-ID: | 43c6ec161f930e385dbc3169a065a917cfc60714.camel@j-davis.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Tue, 2020-12-15 at 17:37 +0000, Simon Riggs wrote:
>
> I definitely don't agree with the premise that all existing heap
> options should be common across all new or extension tableAMs. There
> are dozens of such options and I don't believe that they would all
> have meaning in all future storage plugins.
I agree in principle, but looking at the options that are present
today, a lot of them are integrated into general database features,
like autovacuum, toast, logical replication, and parellel query
planning.
We need to have some answer about how these features should interact
with a custom AM if we can't assume anything about the reloptions it
understands.
> I think options should just work exactly the same for Indexes and
> Tables.
How should that work with the existing code? Should we have separate AM
hooks for each of these interaction points, and then have the AM figure
out how to handle its options? What about the toast.* options?
It feels like some common options would make sense to avoid too much
code duplication.
I am not trying to push this in a specific direction, but I don't have
a lot of good answers, so I went for the simplest thing I could think
of that would allow an extension to have its own options, even if it's
a bit hacky. I'm open to alternatives.
Regards,
Jeff Davis
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2020-12-16 00:12:28 | Re: Proposed patch for key managment |
Previous Message | Justin Pryzby | 2020-12-15 23:58:37 | Re: Allow CLUSTER, VACUUM FULL and REINDEX to change tablespace on the fly |