Re: [COMMITTERS] pgsql: Make pg_dump exclude unlogged table data on hot standby slaves
Andres Freund <andres@2ndquadrant.com>
From: Andres Freund <andres@2ndquadrant.com>
To: Magnus Hagander <magnus@hagander.net>
Cc: pgsql-committers <pgsql-committers@postgresql.org>, pgsql-hackers@postgresql.org
Date: 2013-01-25T12:59:37Z
Lists: pgsql-hackers
Attachments
- 0001-Fix-backpatch-of-unlogged-table-check-for-pg_dump.patch (text/x-patch) patch 0001
On 2013-01-25 13:56:11 +0100, Magnus Hagander wrote: > On Fri, Jan 25, 2013 at 1:31 PM, Andres Freund <andres@2ndquadrant.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. 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. Andres -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services
Commits
-
Make pg_dump exclude unlogged table data on hot standby slaves
- 1cc43979cf44 9.1.8 cited