On Mon, Feb 15, 2016 at 9:11 PM, Stephen Frost <sfrost@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?
Yes
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.
Ok
PFA patch fixing those things.
--
Best Wishes, Ashutosh Bapat EnterpriseDB Corporation The Postgres Database Company