Re: thinko in basic_archive.c

Robert Haas <robertmhaas@gmail.com>

From: Robert Haas <robertmhaas@gmail.com>
To: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Cc: bharath.rupireddyforpostgres@gmail.com, nathandbossart@gmail.com, pgsql-hackers@postgresql.org
Date: 2022-10-19T14:48:03Z
Lists: pgsql-hackers
On Tue, Oct 18, 2022 at 11:28 PM Kyotaro Horiguchi
<horikyota.ntt@gmail.com> wrote:
> > > Yeah, leaving a potentially unbounded number of files around after
> > > system crashes seems pretty unfriendly. I'm not sure how to fix that,
> > > exactly.
>
> Unbounded number of sequential crash-restarts itself is a more serious
> problem..

They don't have to be sequential. Garbage could accumulate over weeks,
months, or years.

Anyway, I agree we should hope that the system doesn't crash often,
but we cannot prevent the system administrator from removing the power
whenever they like. We can however try to reduce the number of
database-related things that go wrong if this happens, and I think we
should. Bharath's patch seems like it's probably a good idea, and if
we can do better, we should.

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



Commits

  1. Fix calculation related to temporary WAL segment name in basic_archive