Re: replay of CREATE TABLESPACE eats data at wal_level=minimal

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Noah Misch <noah(at)leadboat(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: replay of CREATE TABLESPACE eats data at wal_level=minimal
Date: 2021-08-23 13:08:44
Message-ID: CA+TgmobkXKzLe9QDWowUS_3uyDLK+p6dA99D1pYkOqWhQaaSKw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, Aug 22, 2021 at 6:59 PM Noah Misch <noah(at)leadboat(dot)com> wrote:
> Here's what I plan to push. Besides adding a test, I modified things so
> CREATE TABLESPACE redo continues to report an error if a non-directory exists
> under the name we seek to create. One could argue against covering that
> corner case, but TablespaceCreateDbspace() does cover it.

By and large, LGTM, though perhaps it would be appropriate to also
credit me as the reporter of the issue.

I feel it might be slightly better to highlight somewhere, either in
the commit message or in the comments, that removing the old directory
is unsafe, because if wal_level=minimal, we may have no other copy of
the data. For me that's the key point here. I feel that the commit
message and comments inside the patch explain rather thoroughly the
possible consequences of the bug and why this particular fix was
chosen, but they're not real explicit about why there was a bug at
all.

Thanks very much for working on this.

--
Robert Haas
EDB: http://www.enterprisedb.com

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Haas 2021-08-23 13:42:12 Re: .ready and .done files considered harmful
Previous Message Amit Kapila 2021-08-23 12:01:17 Re: [BUG] wrong refresh when ALTER SUBSCRIPTION ADD/DROP PUBLICATION