Re: Cluster::restart dumping logs when stop fails - Mailing list pgsql-hackers

From Daniel Gustafsson
Subject Re: Cluster::restart dumping logs when stop fails
Date
Msg-id 4F7EF0B0-1658-40DC-884C-593581AF9FFE@yesql.se
Whole thread Raw
In response to Re: Cluster::restart dumping logs when stop fails  (Andrew Dunstan <andrew@dunslane.net>)
Responses Re: Cluster::restart dumping logs when stop fails
List pgsql-hackers
> On 7 Apr 2024, at 14:51, Andrew Dunstan <andrew@dunslane.net> wrote:
> On 2024-04-06 Sa 20:49, Andres Freund wrote:

>> That's probably unnecessary optimization, but it seems a tad silly to read an
>> entire, potentially sizable, file to just use the last 1k. Not sure if the way
>> slurp_file() uses seek supports negative ofsets, the docs read to me like that
>> may only be supported with SEEK_END.
>
> We should enhance slurp_file() so it uses SEEK_END if the offset is negative.

Absolutely agree.  Reading the thread I think Andres argues for not printing
anything at all in this case but we should support negative offsets anyways, it
will fort sure come in handy.

--
Daniel Gustafsson




pgsql-hackers by date:

Previous
From: Tomas Vondra
Date:
Subject: Re: BitmapHeapScan streaming read user and prelim refactoring
Next
From: Melanie Plageman
Date:
Subject: Re: BitmapHeapScan streaming read user and prelim refactoring