[HACKERS] A few string fixed

2013-03-20 Thread Daniele Varrazzo
Hello,

while translating the new PostgreSQL 9.3 strings I've found a couple
questionable. Patches attached.

Cheers,

-- Daniele


0001-Fixed-MultiXactIds-string-warning.patch
Description: Binary data


0002-Fixed-pasto-in-hint-string-about-making-views-deleta.patch
Description: Binary data

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] A few string fixed

2013-03-20 Thread Tom Lane
Daniele Varrazzo daniele.varra...@gmail.com writes:
 while translating the new PostgreSQL 9.3 strings I've found a couple
 questionable. Patches attached.

Hmm ... I agree with the MultiXactId-MultiXactIds changes, but not with
this one:

 -errhint(To make the view 
 updatable, provide an unconditional ON DELETE DO INSTEAD rule or an INSTEAD 
 OF DELETE trigger.)));
 +errhint(To make the view 
 deletable, provide an unconditional ON DELETE DO INSTEAD rule or an INSTEAD 
 OF DELETE trigger.)));

We use the phrase updatable view, we don't say deletable view
(and this usage is also found in the SQL standard).  We could possibly
make the message say To make the view updatable in this way, or
... for this purpose, but that seems a bit long-winded to me.

regards, tom lane


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers


Re: [HACKERS] A few string fixed

2013-03-20 Thread Daniele Varrazzo
On Wed, Mar 20, 2013 at 2:01 PM, Tom Lane t...@sss.pgh.pa.us wrote:
 Daniele Varrazzo daniele.varra...@gmail.com writes:
 while translating the new PostgreSQL 9.3 strings I've found a couple
 questionable. Patches attached.

 Hmm ... I agree with the MultiXactId-MultiXactIds changes, but not with
 this one:

 -errhint(To make the view 
 updatable, provide an unconditional ON DELETE DO INSTEAD rule or an INSTEAD 
 OF DELETE trigger.)));
 +errhint(To make the view 
 deletable, provide an unconditional ON DELETE DO INSTEAD rule or an INSTEAD 
 OF DELETE trigger.)));

 We use the phrase updatable view, we don't say deletable view
 (and this usage is also found in the SQL standard).  We could possibly
 make the message say To make the view updatable in this way, or
 ... for this purpose, but that seems a bit long-winded to me.

Ok, I'd just thought it was a pasto.

-- Daniele


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers