From: | "Rader, David" <davidr(at)openscg(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [BUGS] Seems bug in postgres_fdw? |
Date: | 2017-03-06 14:45:47 |
Message-ID: | CAABt7R7zPo-mWKz6UTKOPqXYAc9chq3RzRvvif=m3xk2kveJmg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs pgsql-hackers |
On Sat, Mar 4, 2017 at 12:52 AM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Thu, Mar 2, 2017 at 3:28 AM, Rader, David <davidr(at)openscg(dot)com> wrote:
> > Attached is a doc patch that updates the documentation for postgres-fdw
> to
> > include the actual values for the 4 session variables that are set. Does
> > that make sense to clarify?
>
> From my point of view, this would be a sensible thing to document,
> although I feel like the grammar is not quite right, because after
> "establishes remote session settings for the parameters" my brain
> expects a list of parameters rather than a list of parameters and the
> corresponding values. I think it reads better if you change "for the
> parameters" to "for various parameters".
>
> </nitpicking>
>
Revised doc patch attached with various parameters.
Attachment | Content-Type | Size |
---|---|---|
0001-Document-postgres-fdw-session-settings-for-parameter.patch | text/x-patch | 1.8 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Masahiko Sawada | 2017-03-07 10:55:58 | Re: Two phase commit in ECPG |
Previous Message | Mark Kirkwood | 2017-03-05 03:59:14 | Re: BUG #14577: error:ppas-9.5 service already exists on your system |
From | Date | Subject | |
---|---|---|---|
Next Message | Amos Bird | 2017-03-06 14:51:30 | Re: PATCH: psql show index with type info |
Previous Message | Peter Eisentraut | 2017-03-06 14:38:49 | Re: [COMMITTERS] pgsql: Use asynchronous connect API in libpqwalreceiver |