Re: Minor code improvements to create_foreignscan_plan/ExecInitForeignScan

From: Etsuro Fujita <fujita(dot)etsuro(at)lab(dot)ntt(dot)co(dot)jp>
To: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, Michael Paquier <michael(dot)paquier(at)gmail(dot)com>, PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Minor code improvements to create_foreignscan_plan/ExecInitForeignScan
Date: 2016-02-03 03:41:04
Message-ID: 56B176D0.30504@lab.ntt.co.jp
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2016/02/03 3:31, Alvaro Herrera wrote:
> Etsuro Fujita wrote:
>
>> Done. Attached is an updated version of the patch.
>
> Pushed, thanks.

Thank you!

> I kinda wonder why this struct member has a name that doesn't match the
> naming convention in the rest of the struct, and also why isn't it
> documented in the comment above the struct definition. But that's not
> this patch's fault.

I think so, too.

Best regards,
Etsuro Fujita

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Corey Huinker 2016-02-03 03:42:14 Re: [POC] FETCH limited by bytes.
Previous Message Robert Haas 2016-02-03 03:31:15 Re: Raising the checkpoint_timeout limit