2014/1/12 Marko Tiikkaja <ma...@joh.to>

> On 1/9/14, 2:57 PM, Dean Rasheed wrote:
>
>> Yes, this should just return the number of elements, and 0 for an empty
>> array.
>>
>> How it should behave for multi-dimensional arrays is less clear, but
>> I'd argue that it should return the total number of elements, i.e.
>> cardinality('{{1,2},{3,4}}'::int[][]) = 4. That would make it
>> consistent with the choices we've already made for unnest() and
>> ordinality:
>>   - cardinality(foo) = (select count(*) from unnest(foo)).
>>   - unnest with ordinality would always result in ordinals in the range
>> [1, cardinality].
>>
>
> Ignoring my proposal, this seems like the most reasonable option.  I'll
> send an updated patch along these lines.
>

+1

Pavel


>
>
> Regards,
> Marko Tiikkaja
>

Reply via email to