From: | Alvaro Herrera <alvherre(at)dcc(dot)uchile(dot)cl> |
---|---|
To: | Patches <pgsql-patches(at)postgresql(dot)org> |
Subject: | [subxacts] Fixing TODO items |
Date: | 2004-07-26 20:01:43 |
Message-ID: | 20040726200143.GA11479@dcc.uchile.cl |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-patches |
Hackers,
Here is a patch that fixes some of the TODO items in the nested xacts
list:
handle large objects
password file
namespace (temp namespace cleanup)
files
Make TransactionIdIsInProgress more efficient. Make a subxid cache and
a negative xid cache.
Nesting level report via a ParameterStatus message sent just before
ReadyForQuery (Z message).
Allow a readonly subtransaction inside a read-write parent, but not
viceversa
Several tests in tqual.c have been 'optimized'. This means most uses of
SubTransXidsHaveCommonAncestor are gone. Also there are additional
places where HEAP_XMAX_INVALID is set.
There are some likely controversial changes; the Xid caches, in the
first place. I have tested carefully but I'm not sure if all the
possible race conditions are accounted for.
Second, the storage/file/fd.c management. I unified the arrays for
AllocateDir and AllocateFile in a single array. It works fine AFAICS,
but I'd feel better if someone more knowledgeable takes a look.
Password file was already reviewed -- comments needed some work, which
is in. Temp namespace handling uses the same design idea.
Large objects are managed setting the CurrentResourceOwner to
TopTransactionResourceOwner; they are manually freed on subtransaction
abort (if allocated inside the subtransaction), and kept open till main
transaction commit.
Please review this patch. In the meantime I'll concentrate on some SGML
docs and sourcecode comments where needed.
--
Alvaro Herrera (<alvherre[a]dcc.uchile.cl>)
"I personally became interested in Linux while I was dating an English major
who wouldn't know an operating system if it walked up and bit him."
(Val Henson)
Attachment | Content-Type | Size |
---|---|---|
nested-other-1.patch | text/plain | 70.3 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2004-07-26 20:10:17 | Re: win32 version info |
Previous Message | Magnus Hagander | 2004-07-26 19:53:59 | Re: win32 version info |