Re: per feedback I received directly, pg_dump manpage patch

On 05/27/2015 06:31 PM, Alvaro Herrera wrote:
>
> Joshua D. Drake wrote:
>> The man page does not clearly state that pg_dump will not dump things like
>> users or roles. It does mention briefly in the notes section that pg_dumpall
>> is needed for items that are set via SET but that isn't very clear on the
>> other items.
>
> I agree that we need a warning about this, but I think it should be a
> bit more generic rather than focused exclusively on pg_dumpall; perhaps
> something like "global objs such as roles/tblspcs should be backed up
> separately, for example using pg_dumpall".

As the only way to back up global objects is pg_dumpall....?

>
> I didn't check the location of the warning in your patch, or whether a
> warning is really appropriate rather than, say, <note> or a plain
> <para>.

The warning is directly after the first paragraph. I thought about using
<note> but I think this does warrant a <warning>. You can't actually
restore a database correctly without pg_dump + pg_dumpall (in this context).

JD

--
Command Prompt, Inc. - http://www.commandprompt.com/ 503-667-4564
PostgreSQL Centered full stack support, consulting and development.
Announcing "I'm offended" is basically telling the world you can't
control your own emotions, so everyone else should do it for you.