I think that you need to take a little broader look at this section. > At the top, it says "To use any of these functions, you need to > include the header file foreign/foreign.h in your source file", but > this function is defined in foreign/fdwapi.h. It's not clear to me > whether we should consider moving the prototype, or just document that > this function is someplace else. The other functions prototyped in > fdwapi.h aren't documented at all, except for > IsImportableForeignTable, which is mentioned in passing. > > Further down, the section says "Some object types have name-based > lookup functions in addition to the OID-based ones:" and you propose > to put the documentation for this function after that. But this > comment doesn't actually describe this particular function. >
> Actually, this function just doesn't seem to fit into this section at > all. It's really quite different from the others listed there. How > about something like the attached instead? Right. Mentioning the function in the description of relevant function looks better and avoids some duplication. -- Best Wishes, Ashutosh Bapat EnterpriseDB Corporation The Postgres Database Company
