From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Alvaro Herrera <alvherre(at)dcc(dot)uchile(dot)cl> |
Cc: | Patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | Re: [subxacts] Last try before beta |
Date: | 2004-08-01 17:41:39 |
Message-ID: | 22868.1091382099@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Alvaro Herrera <alvherre(at)dcc(dot)uchile(dot)cl> writes:
> Here is the last patch before beta for subtransactions. It fixes the
> following outstanding issues:
Applied with revisions. I had to whack around the code associated with
the subtrans XID cache, because it was seriously violating the locking
requirements for addition/removal of running XIDs in PGPROC (see the
comments in GetSnapshotData, GetNewTransactionId, CommitTransaction).
It's possible that for subxact XIDs we need not be quite so tense, but
I would rather be conservative now and consider optimizations later.
> - Update comments in code, htup.h and xact.c
The "update" in xact.c seemed to consist of wholesale removal of the top
comment. I didn't apply it, because the comment didn't seem that
useless. It does need editorial work, but it's not content-free...
> - SetReindexProcessing
On this it seemed better to make use of TRY/CATCH to reset the flag
instead of putting the burden on xact.c. Now that we have TRY/CATCH
I think we should make an effort to do things that way where possible.
> Note that I deliberately broke the API for EOXactCallbacks, because
> something that needs to do something at that low a level surely needs to
> be at least revised to check if it's OK regarding subtransactions.
Agreed.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2004-08-01 17:45:57 | Re: String consistency |
Previous Message | Tom Lane | 2004-08-01 15:31:48 | Re: Random regression failures |