Re: [PATCH] parser: optionally warn about missing AS for column and table aliases

Поиск
Список
Период
Сортировка
От Marko Tiikkaja
Тема Re: [PATCH] parser: optionally warn about missing AS for column and table aliases
Дата
Msg-id 540A20EE.7020109@joh.to
обсуждение исходный текст
Ответ на [PATCH] parser: optionally warn about missing AS for column and table aliases  (Oskari Saarenmaa <os@ohmu.fi>)
Ответы Re: [PATCH] parser: optionally warn about missing AS for column and table aliases  (Oskari Saarenmaa <os@ohmu.fi>)
Re: [PATCH] parser: optionally warn about missing AS for column and table aliases  (David G Johnston <david.g.johnston@gmail.com>)
Список pgsql-hackers
On 2014-09-05 22:38, Oskari Saarenmaa wrote:
> I wrote the attached patch to optionally emit warnings when column or table
> aliases are used without the AS keyword after errors caused by typos in
> statements turning unintended things into aliases came up twice this week.
> First in a discussion with a colleague who was surprised by a 1 row result
> for the query 'SELECT COUNT(*) files' and again in the "pl/pgsql 2" thread
> as plpgsql currently doesn't throw an error if there are more result columns
> than output columns (SELECT a b INTO f1, f2).
>
> The patch is still missing documentation and it needs another patch to
> modify all the statements in psql & co to use AS so you can use things like
> \d and tab-completion without triggering the warnings.  I can implement
> those changes if others think this patch makes sense.

I think this is only problematic for column aliases.  I wouldn't want to 
put these two to be put into the same category, as I always omit the AS 
keyword for tables aliases (and will continue to do so), but never omit 
it for column aliases.


.marko



В списке pgsql-hackers по дате отправления:

Предыдущее
От: Oskari Saarenmaa
Дата:
Сообщение: [PATCH] parser: optionally warn about missing AS for column and table aliases
Следующее
От: Oskari Saarenmaa
Дата:
Сообщение: Re: [PATCH] parser: optionally warn about missing AS for column and table aliases