Re: [PATCH][PROPOSAL] Add enum releation option type

From: Michael Paquier <michael(at)paquier(dot)xyz>
To: Nikolay Shaplov <dhyan(at)nataraj(dot)su>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Robert Haas <robertmhaas(at)gmail(dot)com>, Nikita Glukhov <n(dot)gluhov(at)postgrespro(dot)ru>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: [PATCH][PROPOSAL] Add enum releation option type
Date: 2019-02-20 06:08:32
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On Sun, Jan 06, 2019 at 06:28:11PM +0300, Nikolay Shaplov wrote:
> Actually I am not satisfied with it too... That is why I started that bigger
> reloptions refactoring work. So for now I would offer to keep initialization
> as it was for other types: in initialize_reloptions using [type]RelOpts[]
> arrays. And then I will offer patch that changes it for all types and remove
> difference between biult-in reloptions and reloptions in extensions.

Should this be switched as waiting on author or just marked as
returned with feedback then?

> 2.5 May be this src/test/modules dummy index is subject to another patch. So
> I will start working on it right now, but we will do this work not dependent
> to any other patches. And just add there what we need when it is ready. I
> would actually add there some code that checks all types of options, because
> we actually never check that option value from reloptons successfully reaches
> extension code. I did this checks manually, when I wrote that bigger reloption
> patch, but there is no facilities to do regularly check is via test suit.
> Creating dummy index will create such facilities.

bloom is a user-facing extension, while src/test/modules are normally
not installed (there is an exception for MSVC anyway..), so stressing
add_enum_reloption in src/test/modules looks more appropriate to me,
except if you can define an option which is useful for the user and is
an enum.

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Kato, Sho 2019-02-20 06:11:22 RE: Speeding up creating UPDATE/DELETE generic plan for partitioned table into a lot
Previous Message Jamison, Kirk 2019-02-20 06:04:52 RE: Timeout parameters