From: | Andrew Dunstan <andrew(at)dunslane(dot)net> |
---|---|
To: | Joachim Wieland <joe(at)mcknight(dot)de>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | parallel pg_dump |
Date: | 2012-04-03 13:26:52 |
Message-ID: | 4F7AFA9C.4040008@dunslane.net |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
I haven't finished reviewing this yet - but there are some things that
need to be fixed.
First, either the creation of the destination directory needs to be
delayed until all the sanity checks have passed and we're sure we're
actually going to write something there, or it needs to be removed if we
error exit before anything gets written there. Example: if there's an
error because I am dumping a 9.1 server and so should have specified
--no-synchronized-snapshots then getting the directory as a by-product
which I need to remove is annoying. Maybe pg_dump -F d should be
prepared to accept an empty directory as well as a non-existent
directory, just as initdb can. Maybe this isn't directly related to this
patch, but I have noticed it more when reviewing this patch.
Second, all the PrintStatus traces are annoying and need to be removed,
or perhaps better only output in debugging mode (using ahlog() instead
of just printf())
cheers
andrew
From | Date | Subject | |
---|---|---|---|
Next Message | Albe Laurenz | 2012-04-03 13:31:46 | Re: pgsql_fdw, FDW for PostgreSQL server |
Previous Message | Dimitri Fontaine | 2012-04-03 12:29:04 | Re: Command Triggers patch v18 |