Re: CommitFest Status Summary - 2010-02-14

From: Greg Stark <stark(at)mit(dot)edu>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: CommitFest Status Summary - 2010-02-14
Date: 2010-02-17 12:26:03
Message-ID: 407d949e1002170426pde945dcx4571a5c682ebda9e@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Feb 17, 2010 at 6:39 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> * Faster CREATE DATABASE by delaying fsync.  This is really two
>> patches now, one of which is apparently to be backpatched.
>
> This one (both parts) seems to have crashed and burned on unexpected
> portability issues :-(.  Do we have any expectation of being able to
> fix it before alpha4?

The current status is that the patch is in HEAD but the one line
fsyncing the directory is #ifdef'd out.
The backpatched fsync of the directory is entirely reverted from early branches.

However I was just looking at the code and realized it has a file
descriptor leak if an error occurs re-opening the files to fsync them.
But then the same file descriptor leak is there if it has an error
opening the destination file so I guess this isn't a new bug.

--
greg

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tim Bunce 2010-02-17 12:28:28 Re: PostgreSQL::PLPerl::Call - Simple interface for calling SQL functions from PostgreSQL PL/Perl
Previous Message Tim Bunce 2010-02-17 12:14:40 Re: CommitFest Status Summary - 2010-02-14