Re: Reduce lock levels for ADD and DROP COLUMN

From: Simon Riggs <simon(at)2ndQuadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Reduce lock levels for ADD and DROP COLUMN
Date: 2010-12-27 16:04:40
Message-ID: 1293465880.1193.64199.camel@ebony
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, 2010-12-27 at 10:41 -0500, Tom Lane wrote:
> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
> > On Mon, Dec 27, 2010 at 6:42 AM, Simon Riggs <simon(at)2ndquadrant(dot)com> wrote:
> >> Idea is to reduce lock level of ADD/DROP COLUMN from AccessExclusiveLock
> >> down to ShareRowExclusiveLock.
> >>
> >> To make it work, we need to recognise that we are adding a column
> >> without rewriting the table.
>
> > Can you elaborate on why you think that's the right test? It seems to
> > me there could be code out there that assumes that the tuple
> > descriptor won't change under it while it holds an AccessShareLock.
>
> s/could/definitely is/
>
> I think this is guaranteed to break stuff; to the point that I'm
> not even going to review the proposal in any detail.

Our emails crossed.

Do you disagree with the ADD or the DROP, or both?

What "stuff" will break, in your opinion? I'm not asking you to do the
research, but a few curveballs would be enough to end this quickly, and
leave a good record for the archives.

--
Simon Riggs http://www.2ndQuadrant.com/books/
PostgreSQL Development, 24x7 Support, Training and Services

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Alvaro Herrera 2010-12-27 16:12:54 Re: C++ keywords in headers (was Re: [GENERAL] #include <funcapi.h>)
Previous Message Magnus Hagander 2010-12-27 15:55:11 Re: Streaming replication as a separate permissions