Re: nitpick about poor style in MergeAttributes

Поиск
Список
Период
Сортировка
От Alvaro Herrera
Тема Re: nitpick about poor style in MergeAttributes
Дата
Msg-id 20190604201830.GA25150@alvherre.pgsql
обсуждение исходный текст
Ответ на Re: nitpick about poor style in MergeAttributes  (Mark Dilger <hornschnorter@gmail.com>)
Ответы Re: nitpick about poor style in MergeAttributes  (Michael Paquier <michael@paquier.xyz>)
Список pgsql-hackers
On 2019-May-23, Mark Dilger wrote:

> On Thu, May 23, 2019 at 5:24 PM Michael Paquier <michael@paquier.xyz> wrote:

> > Looking closer, this code is not new as of v12.  We have that since
> > e7b3349 which has introduced CREATE TABLE OF.

Yeah, I was not quite understanding why it was being blamed on a commit
that actually *removed* one other callsite that did the same thing.  (I
didn't actually realize at the time that this bug was there, mind.)

> > Anyway, I think that assigning the result of list_delete_cell and
> > adding an assertion like in the attached are saner things to do.

Looks good to me.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



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

Предыдущее
От: Alvaro Herrera
Дата:
Сообщение: Re: No mention of no CIC support for partitioned index in docs
Следующее
От: Andres Freund
Дата:
Сообщение: Re: Binary support for pgoutput plugin