Re: pgsql: Make pg_dump exclude unlogged table data on hot standby slaves

On Fri, Jan 25, 2013 at 8:01 AM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
> On Fri, Jan 25, 2013 at 1:59 PM, Andres Freund <andres(at)2ndquadrant(dot)com> wrote:
>> On 2013-01-25 13:56:11 +0100, Magnus Hagander wrote:
>>> On Fri, Jan 25, 2013 at 1:31 PM, Andres Freund <andres(at)2ndquadrant(dot)com> wrote:
>>> > On 2013-01-25 08:49:10 +0000, Magnus Hagander wrote:
>>> >> Make pg_dump exclude unlogged table data on hot standby slaves
>>> >
>>> > This missed the fact that there is no ExecuteSqlQueryForSingleRow and
>>> > surroundign infrastructure.
>>>
>>> Ugh, that's what I get for pushing something just before getting on a
>>> plane. Thanks for spotting.
>>>
>>> Fix looks good, except I'd just put the query text inline in PQexec()
>>> and not bother with a separate variable..
>>
>> The query needs to get passed to check_sql_result and write_msg as well,
>> thats why I added the extra variable.
>
> Ahh, gotcha.
>
>
>> I don't think my -committers post got through (seems to be restricted),
>> so I am CCing -hackers so somebody else can see the patch.
>
> Ok! I just talked to Robert, and he's taking a look at it for me.

Committed.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company