From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Vik Fearing <vik(at)2ndquadrant(dot)fr> |
Cc: | Ryan Murphy <ryanfmurphy(at)gmail(dot)com>, PostgreSQL DEV mailing list <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Adding type info etc for inheritance errmsg: "child table is missing column ..." |
Date: | 2017-01-07 22:05:02 |
Message-ID: | 10704.1483826702@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Vik Fearing <vik(at)2ndquadrant(dot)fr> writes:
> On 01/07/2017 08:15 PM, Tom Lane wrote:
>> No, and TBH I would vote strongly against including that much detail in
>> this error message anyway. That info could be indefinitely long, and it's
>> not especially relevant to the stated error condition --- for example, the
>> presence of a default is *not* relevant to whether the column matches the
>> parent. I'm okay with shoehorning column type into this message, but not
>> much more than that.
> I agree.
> Perhaps the ERROR message should remain as is, and a DETAIL or HINT line
> could be emitted with the entire column definition (or close to it)?
AFAICS, what Ryan is after would be better served by a separate tool that
would produce a "diff" of the two tables' schemas. Even if we were
willing to overload this error message with everything we know about the
parent column, do you really want to fix discrepancies one column at a
time? And what about properties that can't be uniquely associated with a
single column, such as constraints covering multiple columns?
Also, I have a feeling that a suitable tool is already out there. A
moment's digging in the list archives found this thread with links to
several candidates:
https://www.postgresql.org/message-id/flat/561D27E7.5010906%40trustport.com
and I'm pretty sure there have been other such discussions.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Vik Fearing | 2017-01-07 22:44:01 | Re: Adding type info etc for inheritance errmsg: "child table is missing column ..." |
Previous Message | Vik Fearing | 2017-01-07 19:54:43 | Re: Adding type info etc for inheritance errmsg: "child table is missing column ..." |