On Mon, Feb 15, 2016 at 9:11 PM, Stephen Frost <sfr...@snowman.net> wrote:

> Greetings,
> While getting back into the thread Re: Optimization for updating foreign
> tables in Postgres FDW, I noticed some issues with the docs and
> GetExistingLocalJoinPath():
> GetExistingLocalJoinPath() exists in src/backend/foreign/foreign.c, but
> the docs include discussion of it under 54.2 - Foreign Data Wrapper
> Callback Routines.  Shouldn't this be under 54.3. Foreign Data Wrapper
> Helper Functions?


> Also, the prototype is incorrect in the docs (it
> doesn't return a void)

Thanks for pointing that out.

> and the paragraph about what it's for could do
> with some wordsmithing.

Any specific suggestions?

> A link from 54.2 to 54.3 which mentions it would be fine, of course.


PFA patch fixing those things.

Best Wishes,
Ashutosh Bapat
EnterpriseDB Corporation
The Postgres Database Company

Attachment: geljp_doc.patch
Description: application/download

Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:

Reply via email to