From: | Guillaume Lelarge <guillaume(at)lelarge(dot)info> |
---|---|
To: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Cannot cancel the change of a tablespace |
Date: | 2010-06-21 16:46:36 |
Message-ID: | 4C1F976C.4040802@lelarge.info |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hi,
Today, I tried to cancel the change of a tablespace for a table (ALTER
TABLE ... SET TABLESPACE). I got the "Cancel request sent" but the query
continued and finally succeed. It was a big issue for my customer, and I
wanted to look more into that issue. So, I got a look at the source code
and found we didn't check for interrupts in this part of the code. I
added them, and it seems to work as I wanted.
I added a CHECK_FOR_INTERRUPTS call in the copy_relation_data(),
copy_dir(), and copy_file() functions. Works for me on ALTER TABLE ...
SET TABLESPACE and ALTER DATABASE ... SET TABLESPACE, in 9.0 and 8.4.
Not sure we really want that change, and it don't feel like a bug to me.
Should I add it to to the next commitfest?
Comments?
--
Guillaume
http://www.postgresql.fr
http://dalibo.com
Attachment | Content-Type | Size |
---|---|---|
tbs_interrups_v1.patch | text/x-patch | 1.9 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Robert Haas | 2010-06-21 17:03:19 | Re: deprecating =>, take two |
Previous Message | Robert Haas | 2010-06-21 16:45:01 | Re: beta3 & the open items list |