| From: | Michael Paquier <michael(at)paquier(dot)xyz> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp>, pavel(dot)stehule(at)gmail(dot)com, a(dot)zakirov(at)postgrespro(dot)ru, pgsql-hackers(at)postgresql(dot)org |
| Subject: | Re: pg_get_functiondef forgets about most GUC_LIST_INPUT GUCs |
| Date: | 2018-03-22 01:20:39 |
| Message-ID: | 20180322012038.GB2490@paquier.xyz |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, Mar 21, 2018 at 01:40:23AM -0400, Tom Lane wrote:
> Michael Paquier <michael(at)paquier(dot)xyz> writes:
>> On Tue, Mar 20, 2018 at 01:27:35PM -0400, Tom Lane wrote:
>>> + if (flags & GUC_LIST_QUOTE)
>>> + elog(FATAL, "extensions cannot define GUC_LIST_QUOTE variables");
>
>> This would be better as an ereport with ERRCODE_FEATURE_NOT_SUPPORTED I
>> think. An ERROR is better in my opinion.
>
> I don't mind making it an ereport, but I think it needs to be FATAL
> for the reason stated in the comment.
Okay for the FATAL. I can see that at this time of the day your patch
0002 has already been pushed as 846b5a5 with an elog(). Still, it seems
to me that this is not an internal error but an error caused by an
external cause which can be user-visible, so I would push forward with
switching it to an ereport().
--
Michael
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tomas Vondra | 2018-03-22 01:22:47 | Re: [HACKERS] logical decoding of two-phase transactions |
| Previous Message | Thomas Munro | 2018-03-22 00:36:24 | Re: JIT compiling with LLVM v12.2 |