Re: [BUGS] Seems bug in postgres_fdw?

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

In response to

Responses

Browse pgsql-bugs by date

  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

Browse pgsql-hackers by date

  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