Re: Re: [COMMITTERS] pgsql: Instead of trying (and failing) to allow <<label>> at the end of

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: David Fetter <david(at)fetter(dot)org>
Cc: PG Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Re: [COMMITTERS] pgsql: Instead of trying (and failing) to allow <<label>> at the end of
Date: 2010-03-03 06:00:26
Message-ID: 13586.1267596026@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

David Fetter <david(at)fetter(dot)org> writes:
> On Tue, Mar 02, 2010 at 11:02:54PM -0500, Jaime Casanova wrote:
>> On Tue, Mar 2, 2010 at 8:53 PM, Tom Lane <tgl(at)postgresql(dot)org> wrote:
>>> Instead of trying (and failing) to allow <<label>> at the end of a DECLARE
>>> section, throw an error message saying explicitly that the label must go
>>> before DECLARE.
>>
>> with the docs saying another thing, this couldn't be considered as a
>> bug fix and therefore backpatchable

> +1 for back-patching.

No, I intentionally didn't back-patch that. The only benefit of the
change is throwing a useful error message for questionable syntax.
As against that we have the possibility of breaking code that works
now, if someone's got an incorrectly-placed label that they're not
actually referencing. I grant that that's not a very likely case;
but the error that the message would help people fix isn't very
likely either, as demonstrated by the fact that this never came up
before. We shouldn't be making changes in the behavior of back
branches for such things.

regards, tom lane

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Jaime Casanova 2010-03-03 06:07:11 Re: Re: [COMMITTERS] pgsql: Instead of trying (and failing) to allow <<label>> at the end of
Previous Message David Fetter 2010-03-03 04:59:49 Re: [COMMITTERS] pgsql: Instead of trying (and failing) to allow <<label>> at the end of

Browse pgsql-hackers by date

  From Date Subject
Next Message KaiGai Kohei 2010-03-03 06:03:00 Re: renameatt() can rename attribute of index, sequence, ...
Previous Message A. Kretschmer 2010-03-03 05:55:43 Re: [GENERAL] to_timestamp() and quarters