Re: Allow placeholders in ALTER ROLE w/o superuser

From: Pavel Borisov <pashkin(dot)elfe(at)gmail(dot)com>
To: Alexander Korotkov <aekorotkov(at)gmail(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, Steve Chavez <steve(at)supabase(dot)io>, Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, nathandbossart(at)gmail(dot)com, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Allow placeholders in ALTER ROLE w/o superuser
Date: 2022-12-09 11:52:13
Message-ID: CALT9ZEHgd3JPsQnqs+6DQ+rJjnmQO2yB3kNyqHtQBWZqRNcqgQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi, Alexander!
> Pushed, thanks to everyone!

Thank you!
I've found a minor thing that makes the new test fail on sifaka and
longfin build farm animals. If role names in regression don't start
with "regress_" this invokes a warning. I've consulted in other
modules regression tests e.g. in test_rls_hooks and changed the role
naming accordingly. In essence, a fix is just a batch replace in test
SQL and expected results.

Regards,
Pavel.

Attachment Content-Type Size
0001-Fix-role-names-in-regression-test-in-module-test_pg_.patch application/octet-stream 9.8 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Pavel Borisov 2022-12-09 11:56:05 Re: Allow placeholders in ALTER ROLE w/o superuser
Previous Message Alvaro Herrera 2022-12-09 11:37:44 Re: generic plans and "initial" pruning