Re: Race condition in b-tree page deletion

From: Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>
To: Kevin Grittner <kgrittn(at)ymail(dot)com>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Race condition in b-tree page deletion
Date: 2013-12-17 07:56:09
Message-ID: 52B00399.4020804@vmware.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 12/17/2013 03:57 AM, Kevin Grittner wrote:
> I agree with others that this patch is not suitable for
> back-patching. I wonder whether some other solution might be worth
> back-patching, since it is clearly a bug. The "never delete
> internal pages" might be safe enough to consider. It would lead to
> some degree of bloat, but perhaps bloat is better than errors. Of
> course, the argument can certainly be made that people hit the bug
> so rarely that we're better off just leaving it alone in stable
> branches. Anyway, that would be a different patch.

I don't think "never delete internal pages" would be acceptable. In
particular, it would bloat indefinitely for a smallish FIFO queue table
with a timestamp column.

- Heikki

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message imagenesis@gmail.com 2013-12-17 08:10:47 Re: dpkg-buildpackage fails on 9.2.6 on ubuntu 12.04.3 LTS
Previous Message David Rowley 2013-12-17 07:01:40 Re: [PATCH] Negative Transition Aggregate Functions (WIP)