From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> |
Cc: | michael(at)paquier(dot)xyz, 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-19 23:15:36 |
Message-ID: | 2637.1521501336@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Kyotaro HORIGUCHI <horiguchi(dot)kyotaro(at)lab(dot)ntt(dot)co(dot)jp> writes:
> At Fri, 16 Mar 2018 21:15:54 +0900, Michael Paquier <michael(at)paquier(dot)xyz> wrote in <20180316121554(dot)GA2552(at)paquier(dot)xyz>
>> Let's be clear. I have listed all the variables in the patch to gather
>> more easily opinions, and because it is easier to review the whole stack
>> this way. I personally think that the only variables where the patch
>> makes sense are:
>> - DateStyle
>> - search_path
>> - plpgsql.extra_errors
>> - plpgsql.extra_warnings
>> - wal_consistency_checking
>> So I would be incline to drop the rest from the patch. If there are
>> authors of popular extensions willing to get this support, let's update
>> the list once they argue about it and only if it makes sense. However,
>> as far as I can see, there are no real candidates. So let's keep the
>> list simple.
> FWIW +1 from me. It seems reasonable as the amendment to the
> current status.
It suddenly struck me that the scope of the patch is wider than it needs
to be. We don't need special behavior for all GUC_LIST variables, only
for GUC_LIST_QUOTE variables. (For example, SET datestyle = 'iso, mdy'
works just as well as SET datestyle = iso, mdy.)
This is a good thing not least because all the GUC_LIST_QUOTE variables
are in core. I've been trying to think of a way that we could have
correct behavior for variables that the core backend doesn't know about
and whose extension shlibs aren't currently loaded, and I can't come up
with one. So I think that in practice, we have to document that
GUC_LIST_QUOTE is unsupported for extension variables (and, perhaps,
enforce this in DefineCustomStringVariable? or is that cure worse than
the disease?)
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Langote | 2018-03-20 00:34:23 | Re: inserts into partitioned table may cause crash |
Previous Message | Andres Freund | 2018-03-19 22:22:09 | Re: PostgreSQL 10: Segmentation fault when using GROUPING SETS with all unsortable columns |