From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | ITAGAKI Takahiro <itagaki(dot)takahiro(at)oss(dot)ntt(dot)co(dot)jp> |
Cc: | pgsql-patches(at)postgresql(dot)org, Heikki Linnakangas <heikki(at)enterprisedb(dot)com>, Simon Riggs <simon(at)2ndquadrant(dot)com> |
Subject: | Re: [HACKERS] Fix mdsync never-ending loop problem |
Date: | 2007-04-10 23:50:23 |
Message-ID: | 5021.1176249023@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
I wrote:
> I fooled around with this idea and came up with the attached patch.
> It seems to do what's intended but could do with more eyeballs and
> testing before committing. Comments please?
Earlier I said that I didn't want to back-patch this change, but on
looking at the CVS history I'm reconsidering. The performance problem
originates from the decision some time ago to do an AbsorbFsyncRequests
every so often during the mdsync loop; without that, and assuming no
actual failures, there isn't any absorption of new requests before
mdsync can complete. Originally that code only existed in 8.2.x, but
very recently we back-patched it into 8.1.x as part of fixing the
file-deletion-on-Windows problem. This means that 8.1.x users could
see a performance degradation upon updating to 8.1.8 from prior
subreleases, which wouldn't make them happy.
So I'm now thinking we ought to back-patch into 8.2.x and 8.1.x,
but of course that makes it even more urgent that we test the patch
thoroughly.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Maxime Henrion | 2007-04-10 23:58:55 | Re: [mux@FreeBSD.org: Re: Anyone interested in improving postgresql scaling?] |
Previous Message | Tom Lane | 2007-04-10 22:33:38 | Re: [mux@FreeBSD.org: Re: Anyone interested in improving postgresql scaling?] |
From | Date | Subject | |
---|---|---|---|
Next Message | Koichi Suzuki | 2007-04-11 01:09:43 | Re: [HACKERS] Full page writes improvement, code update |
Previous Message | Tom Lane | 2007-04-10 22:17:14 | Re: Table function support |