Re: [HACKERS] Hash support for grouping sets

>>>>> "Mark" == Mark Dilger <hornschnorter@gmail.com> writes:

 Mark> Reviewing the patch a bit more, I find it hard to understand the
 Mark> comment about passing -1 as a flag for finalize_aggregates.  Any
 Mark> chance you can spend a bit more time word-smithing that code
 Mark> comment?
 
 >> Sure.
 >> 
 >> How does this look for wording (I'll incorporate it into an updated
 >> patch later):

 Mark> Much better.  Thanks!

Here's a version of the patch that obviates that comment entirely, by
moving the differences between the hashed and sorted cases out to the
separate callers of finalize_aggregates. 

-- 
Andrew (irc:RhodiumToad)


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers