On Wed, Dec 2, 2015 at 3:24 PM, Michael Paquier <michael.paqu...@gmail.com> wrote: > On Wed, Dec 2, 2015 at 3:23 PM, Michael Paquier > <michael.paqu...@gmail.com> wrote: >> On Wed, Dec 2, 2015 at 7:05 AM, Tomas Vondra >> <tomas.von...@2ndquadrant.com> wrote: >>> Attached is v2 of the patch, that >>> >>> (a) adds explicit fsync on the parent directory after all the rename() >>> calls in timeline.c, xlog.c, xlogarchive.c and pgarch.c >>> >>> (b) adds START/END_CRIT_SECTION around the new fsync_fname calls >>> (except for those in timeline.c, as the START/END_CRIT_SECTION is >>> not available there) >>> >>> The patch is fairly trivial and I've done some rudimentary testing, but I'm >>> sure I haven't exercised all the modified paths. >> >> I would like to have an in-depth look at that after finishing the >> current CF, I am the manager of this one after all... Could you >> register it to 2016-01 CF for the time being? I don't mind being >> beaten by someone else if this someone has some room to look at this >> patch.. > > And please feel free to add my name as reviewer.
Tomas, I am planning to have a look at that, because it seems to be important. In case it becomes lost on my radar, do you mind if I add it to the 2016-03 CF? -- Michael -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
