Re: Moving more work outside WALInsertLock - Mailing list pgsql-hackers

From Jeff Janes
Subject Re: Moving more work outside WALInsertLock
Date
Msg-id CAMkU=1zT2NOoug6oWZ0tRvkVM74OBzw_GM+ebyRC68nsF1ZM9Q@mail.gmail.com
Whole thread Raw
In response to Re: Moving more work outside WALInsertLock  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Moving more work outside WALInsertLock  (Tom Lane <tgl@sss.pgh.pa.us>)
List pgsql-hackers
On Thu, Dec 15, 2011 at 7:34 AM, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> Heikki Linnakangas <heikki.linnakangas@enterprisedb.com> writes:
>> I've been experimenting with different approaches to do that, but one
>> thing is common among all of them: you need to know the total amount of
>> WAL space needed for the record, including backup blocks, before you
>> take the lock. So, here's a patch to move things around in XLogInsert()
>> a bit, to accomplish that.
>
> This patch may or may not be useful, but this description of it is utter
> nonsense, because we already do compute that before taking the lock.
> Please try again to explain what you're doing?

Currently the CRC of all the data minus the header is computed outside the lock,
but then the header's computation is added and the CRC is finalized
inside the lock.


Cheers,

Jeff


pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: Command Triggers
Next
From: Dimitri Fontaine
Date:
Subject: Re: Command Triggers