Re: Revised Patch to allow multiple table locks in "Unison"

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
Cc: Neil Padgett <npadgett(at)redhat(dot)com>, "pgsql-patches(at)postgresql(dot)org" <pgsql-patches(at)postgresql(dot)org>
Subject: Re: Revised Patch to allow multiple table locks in "Unison"
Date: 2001-08-02 04:18:19
Message-ID: 1684.996725899@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
>> it'd just take a few more processes. The issue is that you are
>> expecting the lock manager to detect or not detect deadlock, when you
>> still have some lock requests up your sleeve that it's not seen yet.
>> As long as you can block before presenting them all, it can never work.

> I know there has been talk about having this done in the lock manager,
> and I know it isn't worth the effort, but I am wondering how you would
> do it even if you were doing in the lock manager with more information
> available.

I'd have to go back and study my 1980's-vintage operating system theory
textbooks before answering that ;-). But acquisition of multiple locks
is a solved problem, AFAIR.

Likely we'd have to throw out the existing lockmanager datastructures
and start fresh, however --- they assume that a proc waits for only one
lock at a time. It'd be a nontrivial bit of work.

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2001-08-02 04:24:37 Re: [PATCHES] Allow IDENT authentication on local connections (Linux only)
Previous Message Bruce Momjian 2001-08-02 04:17:10 Re: Re: What needs to be done?

Browse pgsql-patches by date

  From Date Subject
Next Message Joe Shevland 2001-08-02 04:36:13 RE: Patch to improve commit time performance and a few other things
Previous Message Barry Lind 2001-08-02 04:07:36 Patch to improve commit time performance and a few other things