GetExistingLocalJoinPath() vs. the docs

From: Stephen Frost <sfrost(at)snowman(dot)net>
To: pgsql-hackers(at)postgresql(dot)org
Subject: GetExistingLocalJoinPath() vs. the docs
Date: 2016-02-15 15:41:03
Message-ID: 20160215154103.GN3331@tamriel.snowman.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

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) and the paragraph about what it's for could do
with some wordsmithing.

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

Thanks!

Stephen

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Dmitry Ivanov 2016-02-15 15:44:54 Re: [WIP] ALTER ... OWNER TO ... CASCADE
Previous Message Teodor Sigaev 2016-02-15 15:29:13 Re: [PATCH] we have added support for box type in SP-GiST index