From: | Simon Riggs <simon(at)2ndQuadrant(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Report: race conditions in WAL replay routines |
Date: | 2012-02-06 09:21:02 |
Message-ID: | CA+U5nMJRW8heGCdb8vtmz7pjqPj6OeOPyGJjzgW3yVCp1MrXtg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Sun, Feb 5, 2012 at 11:14 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Simon Riggs <simon(at)2ndQuadrant(dot)com> writes:
>> Please post the patch rather than fixing directly. There's some subtle
>> stuff there and it would be best to discuss first.
>
> And here's a proposed patch for not throwing ERROR during replay of DROP
> TABLESPACE. I had originally thought this would be a one-liner
> s/ERROR/LOG/, but on inspection destroy_tablespace_directories() really
> needs to be changed too, so that it doesn't throw error for unremovable
> directories.
Looks good.
The existing errmsg of "tablespace is not empty" doesn't cover all
reasons why tablespace was not removed.
The final message should have
errmsg "tablespace not fully removed"
errhint "you should resolve this manually if it causes further problems"
The errdetail is good.
--
Simon Riggs http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Heikki Linnakangas | 2012-02-06 10:02:41 | Re: 16-bit page checksums for 9.2 |
Previous Message | Simon Riggs | 2012-02-06 09:13:05 | Re: Report: race conditions in WAL replay routines |