From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Alexey Klyukin <alexk(at)waki(dot)ru> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Jim Nasby <jim(at)nasby(dot)net>, "pgsql-hackers(at)postgresql(dot)org Hackers" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: warn in plperl logs as... NOTICE?? |
Date: | 2010-01-22 16:50:34 |
Message-ID: | 4B59D75A.1060800@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Alexey Klyukin wrote:
> On Jan 22, 2010, at 4:38 PM, Robert Haas wrote:
>
>
>> On Fri, Jan 22, 2010 at 9:13 AM, Alexey Klyukin <alexk(at)waki(dot)ru> wrote:
>>
>>> I think elog(WARNING) is less surprising for the end-user, unless there's an objection strong enough to include it into the documentation :)
>>>
>> I think the main possible objection would what Simon just wrote on the
>> other thread - that it's been this way for a while, and while someone
>> might think that a different decision about how to handle it would
>> have been better, there may be people counting on the current behavior
>> who will have to spend time and perhaps money making changes if we
>> change it.
>>
>
> Well, then we have to choose between a fixed number of unhappy users in the past and potentially increasing number of unhappy users in the future (if we admit the fact that this behavior is illogical). IMO if something behaves counterintuitively to most users the behavior should be at least documented, if not fixed.
>
>
>
Well, as Tim Bunce pointed out, if we get his on_init patch users would
be able to choose which behaviour they wanted. So we don't necessarily
have to choose between what people think conforms to POLA and backwards
compatibility.
Right now I'm a bit hung on that patch because of the "lost GUC
placeholder" issue mentioned elsewhere - everything I have thought of so
far that might overcome it has been unspeakably ugly :-(
cheers
andrew
From | Date | Subject | |
---|---|---|---|
Next Message | Leonardo F | 2010-01-22 16:54:39 | Re: About "Our CLUSTER implementation is pessimal" patch |
Previous Message | Robert Haas | 2010-01-22 16:40:19 | pgsql: Replace ALTER TABLE ... |