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
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 |
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 |