Re: Autovacuum cancellation

From: Simon Riggs <simon(at)2ndquadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Alvaro Herrera <alvherre(at)commandprompt(dot)com>, pgsql-patches(at)postgresql(dot)org
Subject: Re: Autovacuum cancellation
Date: 2007-10-27 22:15:12
Message-ID: 1193523312.4242.623.camel@ebony.site
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers pgsql-patches

On Thu, 2007-10-25 at 17:35 -0400, Tom Lane wrote:
> There's some things still to be desired here: if an autovac process is
> involved in a hard deadlock, the patch doesn't favor zapping it over
> anybody else, nor consider cancelling the autovac as an alternative to
> rearranging queues for a soft deadlock. But dealing with that will
> open cans of worms that I don't think we want to open for 8.3.

I did look at doing that but decided it would not be appropriate to do
that in all cases. i.e. there are hard deadlock cases where the autovac
can be the head of the lock queue and yet a deadlock still exists
between two other processes. The deadlock detector doesn't get called
twice for the same deadlock, so it wasn't possible to speculatively do
that and then re-catch it second time around.

--
Simon Riggs
2ndQuadrant http://www.2ndQuadrant.com

In response to

Browse pgsql-committers by date

  From Date Subject
Next Message User Kostas 2007-10-27 22:17:55 pgtreelib - pgtreelib:
Previous Message User Kostas 2007-10-27 22:06:12 pgtreelib - pgtreelib:

Browse pgsql-hackers by date

  From Date Subject
Next Message Simon Riggs 2007-10-27 22:22:40 Re: Autovacuum cancellation
Previous Message Pavel Stehule 2007-10-27 21:33:34 Re: Proposal: real procedures again (8.4)

Browse pgsql-patches by date

  From Date Subject
Next Message Simon Riggs 2007-10-27 22:22:40 Re: Autovacuum cancellation
Previous Message Hannu Krosing 2007-10-27 21:20:47 Re: [PATCHES] Including Snapshot Info with Indexes