| From: | David Rowley <dgrowleyml(at)gmail(dot)com> |
|---|---|
| To: | Matheus Alcantara <matheusssilv97(at)gmail(dot)com> |
| Cc: | Corey Huinker <corey(dot)huinker(at)gmail(dot)com>, PostgreSQL Developers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Have the planner convert COUNT(1) / COUNT(not_null_col) to COUNT(*) |
| Date: | 2025-11-03 22:47:08 |
| Message-ID: | CAApHDvrWqiq2VVz7-JgNyApQ3y5xr_boBbMgJarAb8db8=mZkA@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Tue, 4 Nov 2025 at 09:38, Matheus Alcantara <matheusssilv97(at)gmail(dot)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
| From | Date | Subject | |
|---|---|---|---|
| Next Message | David Rowley | 2025-11-03 22:49:15 | Re: Have the planner convert COUNT(1) / COUNT(not_null_col) to COUNT(*) |
| Previous Message | Bryan Green | 2025-11-03 22:12:36 | Re: [PATCH] Fix orphaned backend processes on Windows using Job Objects |