Re: fd.c doesn't remove files on a crash-restart

From: "Joshua D(dot) Drake" <jd(at)commandprompt(dot)com>
To: Andres Freund <andres(at)anarazel(dot)de>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: fd.c doesn't remove files on a crash-restart
Date: 2016-03-16 18:02:09
Message-ID: 56E99FA1.80207@commandprompt.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 03/16/2016 10:56 AM, Andres Freund wrote:

>> I understand that this is designed this way. I think it is a bad idea
>> because:
>>
>> 1. The majority crash-restarts in the wild are going to be diagnosed rather
>> easily within the OS itself. They fall into things like OOM killer and out
>> of disk space.
>
> I don't buy 1), like at all. I've seen numerous production instances
> with crashes outside of os triggered things.

I don't argue that. I argue that 9 times out of 10, it will not be one
of those things but ....

>> 3. The problem can get worse over time. If you have a very long running
>> instance, any time the backend crash-restarts you have to potential to
>> increase disk space used for no purpose.
>
> But I think these outweigh the debugging benefit.

Well as Andrew said, we could also create postmaster start option that
defaults to don't save.

JD

--
Command Prompt, Inc. http://the.postgres.company/
+1-503-667-4564
PostgreSQL Centered full stack support, consulting and development.
Everyone appreciates your honesty, until you are honest with them.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2016-03-16 18:02:43 Re: Pushdown target list below gather node (WAS Re: WIP: Upper planner pathification)
Previous Message Robert Haas 2016-03-16 18:00:10 Re: [PROPOSAL] VACUUM Progress Checker.