public inbox for [email protected]  

help / color / mirror / Atom feed

From: Robert Haas <[email protected]>
To: Jaime Casanova <[email protected]>
Cc: Tom Lane <[email protected]>
Cc: Bruce Momjian <[email protected]>
Cc: Scott Marlowe <[email protected]>
Cc: pgsql-docs <[email protected]>
Subject: Re: CREATE USER
Date: Wed, 2 May 2012 13:09:39 -0400
Message-ID: <CA+Tgmoa-PQgqvdQDGriTgyjLMLNoXBzUHkEDGgiX5Y+0f92ztQ@mail.gmail.com> (raw)
In-Reply-To: <CAJKUy5jqgN+qyZj1tG68s8mygNx6_8U1pxPOL6Y=n2FFK5TDnw@mail.gmail.com>
References: <[email protected]>
	<[email protected]>
	<CAJKUy5jqgN+qyZj1tG68s8mygNx6_8U1pxPOL6Y=n2FFK5TDnw@mail.gmail.com>

On Tue, Apr 24, 2012 at 2:55 AM, Jaime Casanova <[email protected]> wrote:
> On Tue, Dec 13, 2011 at 11:27 PM, Tom Lane <[email protected]> wrote:
>> Bruce Momjian <[email protected]> writes:
>>> Well, I know we support the new pg_dump dumping back to 7.1, but I don't
>>> know what our policy is for accepting old dumps.
>>
>> We're still going to very large lengths to load pre-7.3 dumps; see
>> for instance ConvertTriggerToFK() in trigger.c.  I'm not aware of any
>> policy that would suggest blowing off pre-8.1 dumps should be
>> acceptable.  Yeah, it's true that the *recommended* procedure is to
>> use the newer pg_dump, but people aren't always in a position to do
>> that.
>>
>> I think it might be sane to emit a WARNING suggesting that CREATEUSER
>> might not mean what you think, but failing is probably not good.
>>
>
> are we going to do this in this release?
> i never was able to think in a good phrasing for this, though

I actually think we should just leave this alone.  There is a
limitless number of things that someone could potentially be confused
by if they fail to read the documentation, and we can't warn about all
of them.

Now, one thing we could do is add a deprecation warning, stating that
CREATEUSER may be removed in a future release, assuming we want to
eventually remove it.  But I don't think warning people that
CREATEUSER means SUPERUSER and not CREATEROLE is very helpful.

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

view thread (13+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected], [email protected], [email protected], [email protected]
  Subject: Re: CREATE USER
  In-Reply-To: <CA+Tgmoa-PQgqvdQDGriTgyjLMLNoXBzUHkEDGgiX5Y+0f92ztQ@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox