Thread: redundant message?

redundant message?

From
Alvaro Herrera
Date:
I noticed we have this message in utils/adt/misc.c:176 (9.2 branch):
 (errmsg("must be superuser or have the same role to terminate backends running in other server processes"))));

I think the wording "backends running in other server processes" is
redundant.

How about this? (errmsg("must be superuser or have the same role to terminate other server processes"))));

Other ideas are welcome.

--
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services


Re: redundant message?

From
Robert Haas
Date:
On Mon, Aug 6, 2012 at 2:54 PM, Alvaro Herrera <alvherre@2ndquadrant.com> wrote:
> I noticed we have this message in utils/adt/misc.c:176 (9.2 branch):
>
>   (errmsg("must be superuser or have the same role to terminate backends running in other server processes"))));
>
> I think the wording "backends running in other server processes" is
> redundant.

Yeah.

> How about this?
>   (errmsg("must be superuser or have the same role to terminate other server processes"))));

Sounds good to me.

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