|From:||David Steele <david(at)pgmasters(dot)net>|
|To:||Simon Riggs <simon(dot)riggs(at)enterprisedb(dot)com>, Jeff Davis <pgsql(at)j-davis(dot)com>|
|Cc:||Simon Riggs <simon(at)2ndquadrant(dot)com>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>|
|Subject:||Re: Reloptions for table access methods|
|Views:||Raw Message | Whole Thread | Download mbox | Resend email|
On 1/7/21 2:32 PM, Simon Riggs wrote:
> On Sat, Jan 2, 2021 at 6:44 PM Jeff Davis <pgsql(at)j-davis(dot)com> wrote:
>> On Wed, 2020-12-30 at 21:23 +0000, Simon Riggs wrote:
>>> But you cannot seriously argue that a one line patch that changes
>>> visible behavior can be acceptable in Postgres core without tests or
>>> docs or code comments.
>> Often, good documented APIs come about after a few extensions gain
>> experience hacking things together using undocumented assumptions and
>> internal APIs. But in this case, extension authors have no opportunity
>> to hack in reloptions for a TableAM, because of this needless extra
>> check that my patch would eliminate.
>> The patch does not have any user-visible impact. To have any effect, an
>> extension would need to use these internal APIs in a specific way that
>> is not documented externally. I see my tiny patch as a tiny improvement
>> to the backend code because it eliminates a useless extra check, and
>> therefore doesn't need much justification. If others see it as a burden
>> on the engine code, that's a different story.
>> If we insist that this must be a fully-supported feature or nothing at
>> all, then I'll withdraw the patch. But I doubt that will result in a
>> proper feature for v14, it just means that when we do get around to
>> supporting extensible reloptions, there will be no hacks in the wild to
>> learn from.
> Thanks for the reply. I'm trying to get my head around this before a
> longer reply.
Simon, have you had time to formulate your thoughts on this patch?
|Next Message||Pavel Stehule||2021-03-18 14:45:16||Re: pl/pgsql feature request: shorthand for argument and local variable references|
|Previous Message||Masahiko Sawada||2021-03-18 14:30:46||Re: a verbose option for autovacuum|