Re: Surjective functional indexes

From: Konstantin Knizhnik <k(dot)knizhnik(at)postgrespro(dot)ru>
To: Simon Riggs <simon(at)2ndquadrant(dot)com>, Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>
Cc: Christoph Berg <myon(at)debian(dot)org>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Surjective functional indexes
Date: 2017-09-01 08:47:28
Views: Raw Message | Whole Thread | Download mbox | Resend email
Lists: pgsql-hackers

On 01.09.2017 09:25, Simon Riggs wrote:
> On 1 September 2017 at 05:40, Thomas Munro
> <thomas(dot)munro(at)enterprisedb(dot)com> wrote:
>> On Fri, Jun 9, 2017 at 8:08 PM, Konstantin Knizhnik
>> <k(dot)knizhnik(at)postgrespro(dot)ru> wrote:
>>> Attached please find rebased version of the patch.
>>> Now "projection" attribute is used instead of surjective/injective.
>> Hi Konstantin,
>> This still applies but it doesn't compile after commits 2cd70845 and
>> c6293249. You need to change this:
>> Form_pg_attribute att = RelationGetDescr(indexDesc)->attrs[i];
>> ... to this:
>> Form_pg_attribute att = TupleDescAttr(RelationGetDescr(indexDesc), i);
>> Thanks!
> Does the patch work fully with that change? If so, I will review.
Attached please find rebased version of the patch.
Yes, I checked that it works after this fix.
Thank you in advance for review.

Konstantin Knizhnik
Postgres Professional:
The Russian Postgres Company

Attachment Content-Type Size
projection.patch text/x-patch 17.7 KB

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message Masahiko Sawada 2017-09-01 08:59:47 Re: CLUSTER command progress monitor
Previous Message dipesh 2017-09-01 08:42:07 Re: pgjdbc logical replication client throwing exception