Re: multiset patch review

From: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
To: Itagaki Takahiro <itagaki(dot)takahiro(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: multiset patch review
Date: 2011-01-12 12:07:22
Message-ID: AANLkTimTX=OscfZ1_CWUVcGCLAh6R+8JpGsYpQiHOjih@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

2011/1/12 Itagaki Takahiro <itagaki(dot)takahiro(at)gmail(dot)com>:
> On Wed, Jan 12, 2011 at 20:18, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> wrote:
>> there is one issue - probably useless checking a type equality in
>> function check_comparable and check_concatinatable, because when your
>> function is registrated with arguments (anyarray, anyarray), then is
>> guaranteed so type of array1 is same as type of array2, and then you
>> don't need to check.
>
> It's true for almost all cases, but we have "anyarray" columns in
> pg_statistic.stavaluesN. When we pass them to those array functions,
> element types of two anyarrays could be different.
> I guess they are protections only for them.
>
> =# SELECT A.stavalues1 SUBMULTISET OF B.stavalues1
>   FROM pg_statistic A, pg_statistic B
>   WHERE A.stakind1 = 2 AND B.stakind1 = 2;
> ERROR:  cannot compare incompatible arrays
> DETAIL:  Arrays with element types name and oid[] are not compatible
> for comparison.
>

ook

Pavel

> --
> Itagaki Takahiro
>

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Pavel Stehule 2011-01-12 12:10:52 Re: multiset patch review
Previous Message Itagaki Takahiro 2011-01-12 11:35:50 Re: multiset patch review