On Fri, Apr 22, 2016 at 2:32 AM, Robert Haas <robertmhaas@gmail.com> wrote:
> On Wed, Apr 20, 2016 at 3:37 AM, Amit Langote wrote:
>>> I think it'd be better to match the comment with that for
>>> create_foreignscan_path(). So how about "ForeignPath represents a
>>> potential scan of a foreign table, foreign join, or foreign upper-relation
>>> processing"? I think we would probably need to update the comment in
>>> src/backend/optimizer/README (L358), too.
>>
>> That's a lot better. Updated patch attached.
>
> Looks OK to me, too. Committed.
Thanks!
- Amit