Re: BUG #6669: unique index w/ multiple columns and NULLs

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Kevin Grittner <Kevin(dot)Grittner(at)wicourts(dot)gov>
Cc: jo <jose(dot)soares(at)sferacarta(dot)com>, pgsql-bugs(at)postgresql(dot)org, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Subject: Re: BUG #6669: unique index w/ multiple columns and NULLs
Date: 2012-06-04 15:56:23
Message-ID: 20120604155623.GD2352@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

On Mon, Jun 04, 2012 at 10:29:22AM -0500, Kevin Grittner wrote:
> Bruce Momjian <bruce(at)momjian(dot)us> wrote:
>
> > COUNT(*) can't skip nulls because there is no specified column,
> > but why does COUNT(col) skip nulls --- again, inconsistent.
>
> I disagree -- one is counting rows, the other is counting rows with
> a value in that column. I guess one could criticize the syntax for
> specifying that as non-obvious, but it seems pretty reasonable to
> me.

I get your point about COUNT(*) really counting rows, not values, but
why doesn't GROUP BY then skip nulls?

WITH null_test (col1, col2) AS
(
SELECT 1, null
UNION ALL
SELECT null, null
)
SELECT COUNT(*), col2 FROM null_test group by col2
UNION ALL
SELECT COUNT(col1), col2 FROM null_test group by col2;

count | col2
-------+------
2 |
1 |
(2 rows)

Since col2 is null in both places, why it is processed? Looks like
GROUP BY is selecting the NULL rows, then COUNT is processing them based
on its rules.

I think the original complaint is that NULL != NULL in a WHERE clause,
but GROUP BY is able to group them together just fine.

Anyway, just thoughts on the topic.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ It's impossible for everything to be true. +

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Kevin Grittner 2012-06-04 16:26:20 Re: BUG #6669: unique index w/ multiple columns and NULLs
Previous Message Kevin Grittner 2012-06-04 15:29:22 Re: BUG #6669: unique index w/ multiple columns and NULLs