From: | Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Hannes Erven <hannes(at)erven(dot)at>, pgsql-general <pgsql-general(at)postgresql(dot)org> |
Subject: | Re: Unused files in the database directory after crashed VACUUM FULL |
Date: | 2019-02-11 04:03:13 |
Message-ID: | CAEepm=3pjyuTkDaaYnbGObUuAuLR5brB0LVPorE33bjqWeVvKw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-general |
On Mon, Feb 11, 2019 at 10:21 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Ah, so Andrew was correct: we panicked due to lack of WAL space, and
> that explains why the vacuuming process didn't have an opportunity
> to delete the files belonging to the uncommitted new relation.
> It's a pretty well-understood dynamic, I believe. Perhaps we should
> try harder to recover cleanly, but I don't know of anyone putting
> effort into the case.
FTR I am working on a PG13 patch that records relfilenodes of
uncommitted transactions in undo logs, so it can unlink them reliably,
even if you crash (at the cost of introducing a WAL flush before
creating files). I haven't specifically studied the VACUUM FULL case
yet, but in principle this is exactly what my project aims to fix.
It's mostly intended as example code to demonstrate the undo log
machinery (defining undo record types, registering undo log action
functions that are invoked during rollback, rollback of aborted but
not yet rolled back transaction at startup, ...) without having to
understand the whole zheap sandwich at once, but it's also a solution
to an age old problem. More on that soon.
--
Thomas Munro
http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | AJG | 2019-02-11 09:10:40 | Re: tsvector field length limitation |
Previous Message | Tom Lane | 2019-02-10 23:20:59 | Re: Unused files in the database directory after crashed VACUUM FULL |