Re: WIP patch for consolidating misplaced-aggregate checks

Tom Lane <tgl@sss.pgh.pa.us>

View thread

From: Tom Lane <tgl@sss.pgh.pa.us>

To: Alvaro Herrera <alvherre@2ndquadrant.com>

Cc: Robert Haas <robertmhaas@gmail.com>, Pg Hackers <pgsql-hackers@postgresql.org>

Date: 2012-08-09T23:15:26Z

Lists: pgsql-hackers

Attachments

agg-check-refactor-2.patch.gz (application/octet-stream) patch

Here's an updated version taking into account the discussion so far.
It's still a net addition of code (about +200 lines according to
diffstat), but I think the consolidation of logic is probably worth
that.

Any further comments?

			regards, tom lane