Re: GatherMerge misses to push target list

From: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
To: Rushabh Lathia <rushabh(dot)lathia(at)gmail(dot)com>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: GatherMerge misses to push target list
Date: 2017-09-18 09:18:53
Message-ID: CAA4eK1+dabB+GhCysGa9uZ8Kd8TU_+u6GY5JU6Qwk76rCXuyYQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Sep 18, 2017 at 12:46 PM, Rushabh Lathia
<rushabh(dot)lathia(at)gmail(dot)com> wrote:
> Thanks Amit for the patch.
>
> I reviewed the code changes as well as performed more testing. Patch
> looks good to me.
>

In that case, can you please mark the patch [1] as ready for committer in CF app

> Here is the updated patch - where added test-case clean up.
>

oops, missed dropping the function.

Thanks for the review.

[1] - https://commitfest.postgresql.org/15/1293/

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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Arthur Zakirov 2017-09-18 09:39:44 Re: [PATCH] Generic type subscripting
Previous Message Rajkumar Raghuwanshi 2017-09-18 09:13:34 Re: Partition-wise aggregation/grouping