On Thu, Dec 1, 2022 at 6:14 AM Alexander Korotkov <aekorot...@gmail.com> wrote:
> On Wed, Nov 23, 2022 at 1:53 AM Steve Chavez <st...@supabase.io> wrote:
> > So from my side this all looks good!
>
> Thank you for your feedback.
>
> The next revision of the patch is attached.  It contains code
> improvements, comments and documentation.  I'm going to also write
> sode tests.  pg_db_role_setting doesn't seem to be well-covered with
> tests.  I will probably need to write a new module into
> src/tests/modules to check now placeholders interacts with dynamically
> defined GUCs.

Another revision of patch is attached.  It's fixed now that USER SET
values can't be used for PGC_SUSET parameters.  Tests are added.  That
require new module test_pg_db_role_setting to check dynamically
defined GUCs.

------
Regards,
Alexander Korotkov

Attachment: 0001-USER-SET-parameters-for-pg_db_role_setting-v3.patch
Description: Binary data

Reply via email to