Re: Minor comment improvement to create_foreignscan_plan - Mailing list pgsql-hackers

From Robert Haas
Subject Re: Minor comment improvement to create_foreignscan_plan
Date
Msg-id CA+TgmoaC00we+cWDd1BsNHyrYwGA45USJrn0rmFgT7yzBWekRQ@mail.gmail.com
Whole thread Raw
In response to Re: Minor comment improvement to create_foreignscan_plan  (Etsuro Fujita <fujita.etsuro@lab.ntt.co.jp>)
Responses Re: Minor comment improvement to create_foreignscan_plan  (Etsuro Fujita <fujita.etsuro@lab.ntt.co.jp>)
List pgsql-hackers
On Sun, Nov 15, 2015 at 9:25 PM, Etsuro Fujita
<fujita.etsuro@lab.ntt.co.jp> wrote:
> Oops, I've found another one.  I think we should update a comment in
> postgresGetForeignPlan, too; add remote filtering expressions to the list of
> information needed to create a ForeignScan node.

Instead of saying "remote/local", how about saying "remote and local"
or just leaving it out altogether as in the attached?

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

Attachment

pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: Parallel Seq Scan
Next
From: Nikolay Shaplov
Date:
Subject: Re: pageinspect patch, for showing tuple data