Re: PG12 change to DO UPDATE SET column references

Поиск
Список
Период
Сортировка
От James Coleman
Тема Re: PG12 change to DO UPDATE SET column references
Дата
Msg-id CAAaqYe88yOFhHjAS-0RzHkTDmx+NAg=4Km1udnyAkpayiOoVLQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: PG12 change to DO UPDATE SET column references  (Tom Lane <tgl@sss.pgh.pa.us>)
Ответы Re: PG12 change to DO UPDATE SET column references  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
On Sat, Jan 20, 2024 at 12:59 PM Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> James Coleman <jtc331@gmail.com> writes:
> > I do wonder if it's plausible (and sufficiently easy) to improve the
> > error message here. "column 'foo' of relation 'foo'" makes one thing
> > that you've written foo.foo, (in my real-world case the error message
> > also cut off the sql past "foo.", and so I couldn't even tell if the
> > sql was just malformed). At the very least it'd be nice to have a HINT
> > here (perhaps just when the relation and column name match).
>
> > Before I look at where it is, Is such an improvement something we'd be
> > interested in?
>
> A HINT if the bogus column name (1) matches the relation name and
> (2) is field-qualified seems plausible to me.  Then it's pretty
> likely to be a user misunderstanding about whether to write the
> relation name.

Attached is a patch to do just that. We could also add tests for
regular UPDATEs if you think that's useful.

Regards,
James Coleman

Вложения

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

Предыдущее
От: Pavel Stehule
Дата:
Сообщение: Re: Schema variables - new implementation for Postgres 15
Следующее
От: Peter Eisentraut
Дата:
Сообщение: Re: Make documentation builds reproducible