From: | Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Rod Taylor <rbt(at)rbt(dot)ca>, PostgreSQL Patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] Anyone want to assist with the translation of |
Date: | 2002-10-04 02:29:47 |
Message-ID: | 200210040229.g942Tlb27044@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-announce pgsql-general pgsql-hackers pgsql-patches |
Tom's change added to Rod's patch:
Your patch has been added to the PostgreSQL unapplied patches list at:
http://candle.pha.pa.us/cgi-bin/pgpatches
I will try to apply it within the next 48 hours.
---------------------------------------------------------------------------
Tom Lane wrote:
> Rod Taylor <rbt(at)rbt(dot)ca> writes:
> > /* Call CreateComments() to create/drop the comments */
> > CreateComments(ruleoid, classoid, 0, comment);
> > +
> > + heap_close(relation, AccessShareLock);
> > }
> >
> > /*
>
> Ooops.
>
> I think though that this should read
>
> + heap_close(relation, NoLock);
>
> In general, we hold locks on user relations we are modifying until end
> of transaction. This is different from the rule for system catalogs
> (eg, it's okay to drop the AccessShareLock on pg_rewrite a few lines
> above this). The reason for the distinction is that we want to be
> sure that the user relation won't get DROPped by someone else before
> we've committed our changes. (If someone else did try to drop it in
> that interval, they'd not delete the pg_description row we just added,
> because they couldn't see it.) On the other hand, system catalogs such
> as pg_rewrite are not going to go away, by definition, and so it's okay
> to drop their locks early. The only reason we lock system catalogs at
> all is to allow VACUUM FULL to nail down exclusive access to a catalog
> while it vacuums it.
>
> regards, tom lane
>
> ---------------------------(end of broadcast)---------------------------
> TIP 1: subscribe and unsubscribe commands go to majordomo(at)postgresql(dot)org
>
--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Sullivan | 2002-10-04 22:42:27 | PostgreSQL DBA: Toronto, Ontario |
Previous Message | Marc G. Fournier | 2002-10-03 23:50:23 | v7.2.3 Released to fix Potentially Critical Bug |
From | Date | Subject | |
---|---|---|---|
Next Message | Sean Chittenden | 2002-10-04 03:32:19 | Re: schemas in 7.3b1 |
Previous Message | Neil Conway | 2002-10-04 01:55:52 | Re: Core Dump |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2002-10-04 02:45:51 | Re: Improving backend startup interlock |
Previous Message | Bruce Momjian | 2002-10-04 02:29:12 | Re: [HACKERS] pg7.3b1 |
From | Date | Subject | |
---|---|---|---|
Next Message | Dennis Björklund | 2002-10-04 06:27:45 | translation |
Previous Message | Bruce Momjian | 2002-10-04 02:29:12 | Re: [HACKERS] pg7.3b1 |