Re: incremental backup mishandles XLOG_DBASE_CREATE_FILE_COPY

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Noah Misch <noah(at)leadboat(dot)com>
Cc: "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: incremental backup mishandles XLOG_DBASE_CREATE_FILE_COPY
Date: 2024-02-24 10:46:24
Message-ID: CA+TgmoZTD9qu_ihK+TXOUHz9ejj9Ati_4xu0cALfv_k7t1AsBQ@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sat, Feb 24, 2024 at 10:05 AM Noah Misch <noah(at)leadboat(dot)com> wrote:
> Regarding records the summarizer potentially can't ignore that don't deal in
> relfilenodes, these come to mind:
>
> XLOG_DBASE_DROP - covered in this thread's patch
> XLOG_RELMAP_UPDATE
> XLOG_TBLSPC_CREATE
> XLOG_TBLSPC_DROP
> XLOG_XACT_PREPARE

At present, only relation data files are ever sent incrementally; I
don't think any of these touch those.

> Also, any record that writes XIDs needs to update nextXid; likewise for other
> ID spaces. See the comment at "XLOG stuff" in heap_lock_tuple(). Perhaps you
> don't summarize past a checkpoint, making that irrelevant.

I'm not quite following this. It's true that we summarize from one
redo pointer to the next; but also, our summary is only trying to
ascertain which relation data blocks have been modified. Therefore, I
don't understand the relevance of nextXid here.

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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alexander Korotkov 2024-02-24 11:02:01 Re: Removing unneeded self joins
Previous Message Noah Misch 2024-02-24 05:12:05 Re: Removing unneeded self joins