Re: Optimization for updating foreign tables in Postgres FDW - Mailing list pgsql-hackers

On Mon, Apr 11, 2016 at 5:16 PM, Etsuro Fujita
<fujita.etsuro@lab.ntt.co.jp> wrote:
> On 2016/04/11 12:30, Michael Paquier wrote:
>>
>> +                       if ((cancel = PQgetCancel(entry->conn)))
>> +                       {
>> +                           PQcancel(cancel, errbuf, sizeof(errbuf));
>> +                           PQfreeCancel(cancel);
>> +                       }
>> Wouldn't it be better to issue a WARNING here if PQcancel does not
>> succeed?
>
> Seems like a good idea.  Attached is an updated version of the patch.

Thanks for the new version. The patch looks good to me.
-- 
Michael