Horton

Re: Question: test "aggregates" failed in 32-bit machine

David Rowley <dgrowleyml@gmail.com>

View thread View on postgr.es
From: David Rowley <dgrowleyml@gmail.com>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: "Jonathan S. Katz" <jkatz@postgresql.org>, Tomas Vondra <tomas.vondra@enterprisedb.com>, kuroda.hayato@fujitsu.com, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>, John Naylor <john.naylor@enterprisedb.com>, Michael Paquier <michael@paquier.xyz>
Date: 2022-10-02T20:35:55Z
Lists: pgsql-hackers 
On Mon, 3 Oct 2022 at 08:10, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> As attached.

For the master version, I think it's safe just to get rid of
PlannerInfo.num_groupby_pathkeys now.  I only added that so I could
strip off the ORDER BY / DISTINCT aggregate PathKeys from the group by
pathkeys before passing to the functions that rearranged the GROUP BY
clause.

David

Commits

  1. Revert "Optimize order of GROUP BY keys".