| 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:56:05 |
| Message-ID: | CALT9ZEGWn3FymYF9Nc_mRO-NtbJKfh=kk2FMK2+bu_uEykAt5A@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi, Alexander!
> 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.
I see you already pushed the fix for this in beecbe8e5001. So no
worries, it it not needed anymore.
Regards,
Pavel.
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Alexander Korotkov | 2022-12-09 12:30:14 | Re: Allow placeholders in ALTER ROLE w/o superuser |
| Previous Message | Pavel Borisov | 2022-12-09 11:52:13 | Re: Allow placeholders in ALTER ROLE w/o superuser |