Re: pgsql: CREATE INDEX ... INCLUDING (column[, ...]) - Mailing list pgsql-committers

From Tom Lane
Subject Re: pgsql: CREATE INDEX ... INCLUDING (column[, ...])
Date
Msg-id 12282.1460141054@sss.pgh.pa.us
Whole thread Raw
In response to Re: pgsql: CREATE INDEX ... INCLUDING (column[, ...])  (Peter Geoghegan <pg@heroku.com>)
Responses Re: pgsql: CREATE INDEX ... INCLUDING (column[, ...])  (Teodor Sigaev <teodor@sigaev.ru>)
List pgsql-committers
Peter Geoghegan <pg@heroku.com> writes:
> On Fri, Apr 8, 2016 at 11:31 AM, Robert Haas <robertmhaas@gmail.com> wrote:
>> +1.  I wonder if this should be reverted entirely.

> I really wish I could have done more to help with this, but I didn't
> do enough soon enough. Regrettably, I think that the patch just isn't
> ready. For example, the way that expression indexes just aren't
> handled is a cause for concern, as is the general way in which high
> keys are modified during index builds. Interactions with logical
> decoding are also a concern; there could be significant issues there,
> but that analysis just didn't happen. I had significant
> misunderstandings about the patch as recently as this week.

> This should be reverted.

Given those concerns, this *clearly* was not ready to commit.
Please revert, Teodor.

            regards, tom lane


pgsql-committers by date:

Previous
From: Peter Geoghegan
Date:
Subject: Re: pgsql: CREATE INDEX ... INCLUDING (column[, ...])
Next
From: Teodor Sigaev
Date:
Subject: Re: pgsql: CREATE INDEX ... INCLUDING (column[, ...])