Horton

Re: Fix incorrect start up costs for WindowAgg paths (bug #17862)

Nathan Bossart <nathandbossart@gmail.com>

View thread View on postgr.es
From: Nathan Bossart <nathandbossart@gmail.com>
To: David Rowley <dgrowleyml@gmail.com>
Cc: Andy Fan <zhihui.fan1213@gmail.com>, PostgreSQL Developers <pgsql-hackers@lists.postgresql.org>, Tim Palmer <tim3sp@gmail.com>
Date: 2023-08-03T23:54:03Z
Lists: pgsql-hackers 
On Fri, Aug 04, 2023 at 09:28:51AM +1200, David Rowley wrote:
> Thank you for reviewing.  I've pushed the patch to master only.

I'm seeing some reliable test failures for 32-bit builds on cfbot [0].  At
a glance, it looks like the relations are swapped in the plan.

[0] https://api.cirrus-ci.com/v1/artifact/task/5728127981191168/testrun/build-32/testrun/regress/regress/regression.diffs

-- 
Nathan Bossart
Amazon Web Services: https://aws.amazon.com

Commits

  1. Minor adjustments to WindowAgg startup cost code

  2. Account for startup rows when costing WindowAggs