| From: | Stephen Frost <sfrost(at)snowman(dot)net> |
|---|---|
| To: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
| Cc: | "Brightwell, Adam" <adam(dot)brightwell(at)crunchydatasolutions(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: superuser() shortcuts |
| Date: | 2014-10-27 15:40:31 |
| Message-ID: | 20141027154030.GU28859@tamriel.snowman.net |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
* Alvaro Herrera (alvherre(at)2ndquadrant(dot)com) wrote:
> > As I started looking at this, there are multiple other places where
> > these types of error messages occur (opclasscmds.c, user.c,
> > postinit.c, miscinit.c are just a few), not just around the changes in
> > this patch. If we change them in one place, wouldn't it be best to
> > change them in the rest? If that is the case, I'm afraid that might
> > distract from the purpose of this patch. Perhaps, if we want to
> > change them, then that should be submitted as a separate patch?
>
> Yeah. I'm just saying that maybe this patch should adopt whatever
> wording we agree to, not that we need to change other places. On the
> other hand, since so many other places have adopted the different
> wording, maybe there's a reason for it and if so, does anybody know what
> it is. But I have to say that it does look inconsistent to me.
Updated patch attached. Comments welcome.
Thanks!
Stephen
| Attachment | Content-Type | Size |
|---|---|---|
| superuser_cleanup.patch | text/x-diff | 29.4 KB |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2014-10-27 15:41:44 | Re: TAP test breakage on MacOS X |
| Previous Message | Alvaro Herrera | 2014-10-27 15:32:59 | Re: ltree::text not immutable? |