Re: Parallel grouping sets

On Sat, Jan 25, 2020 at 4:22 AM Jesse Zhang <sbjesse(at)gmail(dot)com> wrote:
>
> On Thu, Jan 23, 2020 at 2:47 AM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> >
> > On Sun, Jan 19, 2020 at 2:23 PM Richard Guo <riguo(at)pivotal(dot)io> wrote:
> > >
> > > I realized that there are two patches in this thread that are
> > > implemented according to different methods, which causes confusion.
> > >
> >
> > Both the idea seems to be different. Is the second approach [1]
> > inferior for any case as compared to the first approach? Can we keep
> > both approaches for parallel grouping sets, if so how? If not, then
> > won't the code by the first approach be useless once we commit second
> > approach?
> >
> >
> > [1] - https://www.postgresql.org/message-id/CAN_9JTwtTTnxhbr5AHuqVcriz3HxvPpx1JWE--DCSdJYuHrLtA%40mail.gmail.com
> >
>
> I glanced over both patches. Just the opposite, I have a hunch that v3
> is always better than v5.
>

This is what I also understood after reading this thread. So, my
question is why not just review v3 and commit something on those lines
even though it would take a bit more time. It is possible that if we
decide to go with v5, we can make it happen earlier, but later when we
try to get v3, the code committed as part of v5 might not be of any
use or if it is useful, then in which cases?

--
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com