Re: pg_restore ignores PGDATABASE

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, Erik Rijkers <er(at)xs4all(dot)nl>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: pg_restore ignores PGDATABASE
Date: 2012-08-27 19:27:19
Message-ID: 20120827192719.GU11088@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, Feb 19, 2012 at 10:25:55AM -0500, Andrew Dunstan wrote:
>
>
> On 02/19/2012 08:02 AM, Robert Haas wrote:
> >On Sun, Feb 19, 2012 at 1:18 AM, Erik Rijkers<er(at)xs4all(dot)nl> wrote:
> >>On Sun, February 19, 2012 06:27, Robert Haas wrote:
> >>>On Sat, Feb 18, 2012 at 11:58 AM, Erik Rijkers<er(at)xs4all(dot)nl> wrote:
> >>>>pg_restore ignores environment variable PGDATABASE.
> >>>What exactly do you mean by "ignores"? pg_restore prints results to
> >>>standard output unless a database name is specified. AFAIK, there's
> >>>no syntax to say "I want a direct-to-database restore to whatever you
> >>>think the default database is".
> >>That's right, and that seems contradictory with:
> >>
> >>"This utility [pg_restore], like most other PostgreSQL utilities, also uses the environment
> >>variables supported by libpq (see Section 31.13)."
> >>
> >>as pg_restore does 'ignore' (for want of a better word) PGDATABASE.
> >>
> >>But I think I can conclude from your reply that that behaviour is indeed intentional.
> >It is, because we want there to be a way of converting a custom or tar
> >format archive back to text. I think that probably works out for the
> >best anyway, since pg_restore is a sufficiently dangerous operation
> >that you want to be darn sure you're not doing it on the wrong
> >database. dropdb also requires a database name, while createdb does
> >not, for similar reasons...
>
> Right, I think we probably need to adjust the docs slightly to match
> this reality.

Done, with the attached patch.

--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com

+ It's impossible for everything to be true. +

Attachment Content-Type Size
env.diff text/x-diff 813 bytes

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2012-08-27 19:28:12 Re: foreign key locks
Previous Message Dean Rasheed 2012-08-27 19:26:11 Re: Proof of concept: auto updatable views