Re: BUG #15589: Due to missing wal, restore ends prematurely and opens database for read/write

From: Jeff Janes <jeff(dot)janes(at)gmail(dot)com>
To: leif(at)lako(dot)no, pgsql-bugs(at)lists(dot)postgresql(dot)org
Subject: Re: BUG #15589: Due to missing wal, restore ends prematurely and opens database for read/write
Date: 2019-01-11 14:03:04
Message-ID: CAMkU=1ym=UYLVS37NNSE547Uw0_xPJUuBmyZPqEHARd=Un6eiA@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

On Fri, Jan 11, 2019 at 4:08 AM PG Bug reporting form <
noreply(at)postgresql(dot)org> wrote:

>
> PostgreSQL should have paused recovery also on the first scenario. Then I
> could have added missing wal and continued with restore.
>

I agree with you that something here is not very user friendly. But the
counter argument is that you should not be hiding WAL files from the system
in the first place. Once it is told that the file doesn't exist, it
doesn't make sense to pause because non-existence is usually permanent, so
there is nothing to be done after a pause. Or in other words, the pause
feature is to let you change your mind about what time point you want to
recover to (moving it only forward), not to let you change your mind about
what WAL files exist in the first place. So I don't think this is a bug,
but perhaps there is room for improvement.

At the least, I think we should log an explicit WARNING if the WAL stream
ends before the specified PIT is reached.

Cheers,

Jeff

In response to

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message PG Bug reporting form 2019-01-11 15:25:56 BUG #15590: crosstab_hash unable to work with modified types (CreateTupleDescCopy returning dropped attributes?)
Previous Message PG Bug reporting form 2019-01-11 08:36:35 BUG #15589: Due to missing wal, restore ends prematurely and opens database for read/write

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Eisentraut 2019-01-11 14:09:45 Re: port of INSTALL file generation to XSLT
Previous Message Nikita Glukhov 2019-01-11 13:01:51 Re: [PATCH] kNN for btree