Re: nitpick about poor style in MergeAttributes - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: nitpick about poor style in MergeAttributes
Date
Msg-id 20190604201830.GA25150@alvherre.pgsql
Whole thread Raw
In response to Re: nitpick about poor style in MergeAttributes  (Mark Dilger <hornschnorter@gmail.com>)
Responses Re: nitpick about poor style in MergeAttributes
List 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 by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: No mention of no CIC support for partitioned index in docs
Next
From: Andres Freund
Date:
Subject: Re: Binary support for pgoutput plugin