Re: Unclear error message
Michael Paquier <michael@paquier.xyz>
From: Michael Paquier <michael@paquier.xyz>
To: Laurenz Albe <laurenz.albe@cybertec.at>
Cc: pgsql-hackers@postgresql.org
Date: 2018-09-21T05:17:07Z
Lists: pgsql-hackers
On Thu, Sep 20, 2018 at 09:45:09PM +0200, Laurenz Albe wrote:
> That message is wrong, because "rel" and not "pkrel" is the partitioned table.
> I think it should be
>
> ereport(ERROR,
> (errcode(ERRCODE_WRONG_OBJECT_TYPE),
> errmsg("foreign key cannot be defined on ONLY \"%s\" for a partitioned table",
> Relatio
> nGetRelationName(rel))));
Hmm... There are no test cases for this error message, nor for the one
below "cannot add NOT VALID foreign key to relation foo", which is a bad
idea.
The referenced relation has to be RELKIND_RELATION. Here is another
idea of error message:
cannot use ONLY on partitioned table "foo" with foreign key
--
Michael
Commits
-
Improve two error messages related to foreign keys on partitioned tables
- c8ed820c68bc 11.0 landed
- 9c2a970d1fa4 12.0 landed