On Thu, Nov 27, 2008 at 2:35 PM, Gianni Ciolli
> On Wed, Nov 26, 2008 at 09:30:22AM +0000, Dave Page wrote:
>> Could you look at the posix_fadvise patch please?
> Hi Dave,
> if I'm correct, it looks like the posix_fadvise patch has been already
> reviewed some days ago, the reviewer made some suggestions, and the
> author is examining them.
> Perhaps you meant another patch? Or you are suggesting that there
> should be a multiple/collective review for that patch?
Multiple reviews. The existing review was largely a code style review,
but didn't cover any sort of perf testing etc. Plus it's always good
to get a second opinion :-)
The update to the patch should be posted today, but that is just a few
lines to remove a configure test I'm told. The server code will be
EnterpriseDB UK: http://www.enterprisedb.com
In response to
pgsql-rrreviewers by date
|Next:||From: Gregory Stark||Date: 2008-11-27 19:00:04|
|Subject: Re: new patch|
|Previous:||From: Gianni Ciolli||Date: 2008-11-27 14:35:50|
|Subject: new patch (was Re: Status check: Ibrar, Gianni, Matthew)|