From: | Chris Bandy <bandy(dot)chris(at)gmail(dot)com> |
---|---|
To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: [PATCH] Add schema and table names to partition error |
Date: | 2020-03-19 14:51:15 |
Message-ID: | a31fe66b-3adf-2ac4-2055-6c16bedd1302@gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 3/18/20 11:46 PM, Amit Kapila wrote:
> On Thu, Mar 19, 2020 at 3:55 AM Chris Bandy <bandy(dot)chris(at)gmail(dot)com> wrote:
>>
>>
>> Sorry for these troubles. Attached are patches created using `git
>> format-patch -n -v6` on master at 487e9861d0.
>>
>
> No problem. I have extracted your code changes as a separate patch
> (see attached) as I am not sure we want to add tests for these cases.
Patch looks good.
My last pitch to keep the tests: These would be the first and only
automated tests that verify errtable, errtableconstraint, etc.
> This doesn't apply in back-branches, but I think that is small work
> and we can do that if required.
It looks like the only failing hunk on REL_12_STABLE is in tablecmds.c.
The ereport is near line 5090 there. The partition code has changed
quite a bit compared the older branches. ;-)
Thanks,
Chris
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Cramer | 2020-03-19 14:52:31 | Re: JDBC prepared insert and X00 and SQL_ASCII |
Previous Message | Masahiko Sawada | 2020-03-19 14:42:36 | Re: Internal key management system |