On 5/3/07, Tom Lane <[EMAIL PROTECTED]> wrote:

NikhilS <[EMAIL PROTECTED]> writes:
> Hi Neil,
>> * the patch is broken for expressional indexes, and silently omits
>> copying the predicate that may be associated with an index.

> Since this patch is only supposed to copy unique/primary indexes, I dont
> think we will ever have predicates associated to such indexes?

Huh?  I would expect a clause "INCLUDING INDEXES" to mean copying *all*
indexes.  A clause "INCLUDING CONSTRAINTS" would reasonably act as you
suggest, ie copy only indexes derived from SQL constraint clauses.

But this patch is not for the "INCLUDING INDEXES" case. As mentioned by
Bruce earlier in this thread, this patch is for the following TODO:

      o Have WITH CONSTRAINTS also create constraint indexes


                regards, tom lane

EnterpriseDB               http://www.enterprisedb.com

Reply via email to