From: | Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com> |
---|---|
To: | Kevin Grittner <Kevin(dot)Grittner(at)wicourts(dot)gov> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: SSI SLRU low-level functions first cut |
Date: | 2011-01-02 08:56:21 |
Message-ID: | 4D203DB5.2030707@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 01.01.2011 23:21, Kevin Grittner wrote:
> I've got low-level routines coded for interfacing predicate.c to SLRU
> to handle old committed transactions, so that SSI can deal with
> situations where a large number of transactions are run during the
> lifetime of a single serializable transaction. I'm not actually
> *using* these new functions yet, but that's what I do next. I would
> love it if someone could review this commit and let me know whether
> it looks generally sane.
>
> http://git.postgresql.org/gitweb?p=users/kgrittn/postgres.git;a=commitdiff;h=00a0bc6c47c8173e82e5927d9b75fe570280860f
Nothing checking for the hi-bit flag AFAICS. I guess the code that uses
that would do check it. But wouldn't it be simpler to mark the unused
slots with zero commitseqno, instead of messing with the hi-bit in valid
values?
It's probably not necessary to explicitly truncate the slru at startup.
We don't do that for pg_subtrans, which also doesn't survive restarts.
The next checkpoint will truncate it.
It would possibly be simpler to not reset headXid and tailXid to
InvalidTransactionId when the "window" is empty, but represent that as
tailXid == headXid + 1.
OldSerXidGetMinConflictCommitSeqNo() calls LWLockRelease twice.
--
Heikki Linnakangas
EnterpriseDB http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Stefan Kaltenbrunner | 2011-01-02 09:20:39 | Re: Sync Rep Design |
Previous Message | Heikki Linnakangas | 2011-01-02 08:35:07 | Re: Sync Rep Design |