From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
Cc: | keith(dot)fiske(at)crunchydata(dot)com, pgsql-bugs(at)lists(dot)postgresql(dot)org |
Subject: | Re: BUG #15865: ALTER TABLE statements causing "relation already exists" errors when some indexes exist |
Date: | 2019-06-21 01:54:46 |
Message-ID: | 28827.1561082086@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs pgsql-hackers |
I wrote:
>> Yeah, obviously I fat-fingered something there. Looking ...
After further review it seems like I was led into this error by a siren
singing something about how we could skip collecting the index definition
string for an index we were going to ignore later. (Cue standard lecture
about premature optimization...) That absolutely *does not* work, because
we might not find out till we're considering some later ALTER TYPE
subcommand that the index depends on a relevant constraint. And we have
to capture the index definition before we alter the type of any column it
depends on, or pg_get_indexdef_string will get very confused. That little
dependency wasn't documented anywhere. I also found a pre-existing
comment that contradicted the new reality but I'd missed removing in
e76de8861.
Here's a patch against HEAD --- since I'm feeling more mortal than usual
right now, I'll put this out for review rather than just pushing it.
It might be easier to review the code changes by just ignoring e76de8861
and diffing against tablecmds.c from before that, as I've done in the
second attachment.
BTW, has anyone got an explanation for the order in which psql is
listing the indexes of "anothertab" in this test case?
regards, tom lane
Attachment | Content-Type | Size |
---|---|---|
fix-alter-table-some-more.patch | text/x-diff | 7.3 KB |
code-diff-since-before-e76de8861.patch | text/x-diff | 2.9 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Steve Komarov | 2019-06-21 02:41:35 | Function pg_database_size fails with "Permission denied" on a corrupted fsm file |
Previous Message | Tom Lane | 2019-06-21 00:48:33 | Re: BUG #15827: Unable to connect on Windows using pg_services.conf using Python psycopg2 |
From | Date | Subject | |
---|---|---|---|
Next Message | RekGRpth | 2019-06-21 04:49:10 | Re: Disconnect from SPI manager on error |
Previous Message | Michael Paquier | 2019-06-21 00:45:54 | Re: BUG #15865: ALTER TABLE statements causing "relation already exists" errors when some indexes exist |