On Wed, 2007-07-18 at 16:44 +0100, Heikki Linnakangas wrote: > Simon Riggs wrote: > > Here's the latest version. I've reviewed this to check that this does > > what I want it to do, re-written various comments and changed a few > > minor points in the code. > > > > I've also added a chunk to transam/README that describes the workings of > > the patch from a high level. > > > > Now ready for final review. > > Just a couple of quick comments: > > Having to call XLogBackgroundFlush(2) to flush the WAL up to the latest > async commit is a weird interface. How about passing a "bool force" > argument instead? Or adding another function "XLogFlushCommits". > > walwriter.h is missing the > > #ifndef XXXX_H > #define XXXX_H > > boilerplate.
OK. Will do, thanks. -- Simon Riggs EnterpriseDB http://www.enterprisedb.com ---------------------------(end of broadcast)--------------------------- TIP 6: explain analyze is your friend