Horton

Re: Have the planner convert COUNT(1) / COUNT(not_null_col) to COUNT(*)

David Rowley <dgrowleyml@gmail.com>

View thread View on postgr.es
From: David Rowley <dgrowleyml@gmail.com>
To: Matheus Alcantara <matheusssilv97@gmail.com>
Cc: Corey Huinker <corey.huinker@gmail.com>, PostgreSQL Developers <pgsql-hackers@lists.postgresql.org>
Date: 2025-11-03T22:47:08Z
Lists: pgsql-hackers 
On Tue, 4 Nov 2025 at 09:38, Matheus Alcantara <matheusssilv97@gmail.com> wrote:
> I looked the code and it seems to be in a good shape, but I tried to
> apply the v2 on top of e7ccb247b38 in master to run some tests and a
> rebase is necessary.

Are you sure you've not got something else in your branch? It applies
ok here, and the CFbot isn't complaining either. CFBot's is based on
cf8be0225, which is 2 commits before the one you're trying, but
src/test/regress/expected/aggregates.out hasn't been changed since
2025-10-07.

David

Commits

  1. Have the planner replace COUNT(ANY) with COUNT(*), when possible