From: | "Godfrin, Philippe E" <Philippe(dot)Godfrin(at)nov(dot)com> |
---|---|
To: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, David Rowley <dgrowleyml(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Nagaraj Raj <nagaraj(dot)sf(at)yahoo(dot)com>, Pg Bugs <pgsql-bugs(at)postgresql(dot)org> |
Subject: | RE: [EXTERNAL] Re: ERROR: insufficient columns in the PRIMARY KEY constraint definition |
Date: | 2020-09-30 13:31:35 |
Message-ID: | SA0PR15MB3933EFB54C75C63B490CAEBA82330@SA0PR15MB3933.namprd15.prod.outlook.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs |
I am curious why this is considered Class 0A, versus 42 (syntax error?)
From: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
Sent: Tuesday, September 29, 2020 6:22 PM
To: David Rowley <dgrowleyml(at)gmail(dot)com>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>; Nagaraj Raj <nagaraj(dot)sf(at)yahoo(dot)com>; Pg Bugs <pgsql-bugs(at)postgresql(dot)org>
Subject: [EXTERNAL] Re: ERROR: insufficient columns in the PRIMARY KEY constraint definition
On 2020-Sep-30, David Rowley wrote:
> I didn't go with the same wording. The reason was that I didn't feel
> the word "constraint" had to be mentioned twice.
>
> I won't object if you or Alvaro want to keep Alvaro's suggestion though.
*Shrug* this seems good enough. A purist could complain that it is
redundant, but in practice it's not important.
Here's the proposed error message fix, using the wording that saves
$1.99. I agree that trying to cram the constraint type in the primary
message is uglier.
--
Álvaro Herrera https://www.2ndQuadrant.com/<https://www.2ndQuadrant.com>
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2020-09-30 14:16:33 | Re: [EXTERNAL] Re: ERROR: insufficient columns in the PRIMARY KEY constraint definition |
Previous Message | hubert depesz lubaczewski | 2020-09-30 13:14:27 | Re: BUG #16640: Documentation table formatting issue |