Re: Final Patch for GROUPING SETS

From: Noah Misch <noah(at)leadboat(dot)com>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: Andrew Gierth <andrew(at)tao11(dot)riddles(dot)org(dot)uk>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Greg Stark <stark(at)mit(dot)edu>, Jim Nasby <Jim(dot)Nasby(at)BlueTreble(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Josh Berkus <josh(at)agliodbs(dot)com>, Greg Sabino Mullane <greg(at)turnstep(dot)com>, Marti Raudsepp <marti(at)juffo(dot)org>, Tomas Vondra <tv(at)fuzzy(dot)cz>, Atri Sharma <atri(dot)jiit(at)gmail(dot)com>, Svenne Krap <svenne(at)krap(dot)dk>
Subject: Re: Final Patch for GROUPING SETS
Date: 2015-05-14 06:51:42
Message-ID: 20150514065142.GB3707771@tornado.leadboat.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Thu, May 14, 2015 at 08:38:07AM +0200, Andres Freund wrote:
> On 2015-05-14 02:32:04 -0400, Noah Misch wrote:
> > On Thu, May 14, 2015 at 07:50:31AM +0200, Andres Freund wrote:
> > > Andrew, is that a structure you could live with, or not?
> > >
> > > Others, what do you think?
> >
> > Andrew and I discussed that very structure upthread:
>
> > http://www.postgresql.org/message-id/87d26zd9k8.fsf@news-spur.riddles.org.uk
>
> I don't really believe that that'd necesarily be true. I think if done
> like I sketched it'll likely end up being simpler than the currently
> proposed code. I also don't see why this would make combining hashing
> and sorting any more complex than now. If anything the contrary.
>
> > http://www.postgresql.org/message-id/20141231085845.GA2148306@tornado.leadboat.com
> > http://www.postgresql.org/message-id/20141231210553.GB2159277@tornado.leadboat.com
> >
> > I still believe the words I wrote in my two messages cited.
>
> I.e. that you think it's a sane approach, despite the criticism?

Yes. I won't warrant that it proves better, but it looks promising. Covering
hash aggregation might entail a large preparatory refactoring of nodeHash.c,
but beyond development cost I can't malign that.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2015-05-14 06:59:45 Re: Final Patch for GROUPING SETS
Previous Message Seçkin Alan 2015-05-14 06:44:32 Re: pgAdmin4 Bug fix or my Fault ?