From: | Anastasia Lubennikova <a(dot)lubennikova(at)postgrespro(dot)ru> |
---|---|
To: | pgsql-committers(at)postgresql(dot)org |
Subject: | Re: pgsql: CREATE INDEX ... INCLUDING (column[, ...]) |
Date: | 2016-04-08 18:59:58 |
Message-ID: | 1460141998856-5897721.post@n5.nabble.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers pgsql-hackers |
Peter Geoghegan-3 wrote
> On Fri, Apr 8, 2016 at 11:31 AM, Robert Haas <
> robertmhaas@
> > wrote:
>>> I assume that the problem here is larger than just failure to adhere to
>>> C89 comment style. Was this patch really ready to commit? I'm not very
>>> happy that such a large patch went from "Needs review" to "Committed" in
>>> the blink of an eye on the very last commitfest day ... and artifacts
>>> like
>>> this aren't doing anything to increase my confidence in it.
It was just missed comment after the fix between patch revisions.
> 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.
The answer to the question about expressions is quite simple - they are not
supported by index-only scan, so having them in covering index now is just
wasting of disc space.
It's sad to hear that patch should be reverted. But, of course, I can't
argue with community's decision.
--
View this message in context: http://postgresql.nabble.com/pgsql-CREATE-INDEX-INCLUDING-column-tp5897653p5897721.html
Sent from the PostgreSQL - committers mailing list archive at Nabble.com.
From | Date | Subject | |
---|---|---|---|
Next Message | Stephen Frost | 2016-04-08 19:00:51 | pgsql: In dumpTable, re-instate the skipping logic |
Previous Message | Teodor Sigaev | 2016-04-08 18:53:48 | pgsql: Revert CREATE INDEX ... INCLUDING ... |
From | Date | Subject | |
---|---|---|---|
Next Message | Stas Kelvich | 2016-04-08 19:03:48 | Fwd: Speedup twophase transactions |
Previous Message | Stas Kelvich | 2016-04-08 18:57:59 | Re: Speedup twophase transactions |