Simon Riggs wrote:

> > This simple patch lets someone specifies the xlog directory at initdb
> > time. It uses symlinks to do it, and create and/or set permissions at
> > the directory as appropriate.
> 
> Looks good.
> 
> I'm no expert on coding style, so I won't comment on anything there.
> 
> Any test harness changes? Or a test script?
> 
Unfortunately, nope. :( I tested it manually.
How did i test it?

1) xlog directory doesn't exist -- OK
2) xlog directory exists -- OK
3) xlog directory exists but doesn't have the right permissions --
FAILED
4) xlog diretory exists but is not empty -- FAILED
5) interrupting initdb execution (Ctrl-C) to test the cleanup code -- OK

PS> i didn't test it in a win32 box but it should be ok.

-- 
  Euler Taveira de Oliveira
  http://www.timbira.com/


---------------------------(end of broadcast)---------------------------
TIP 6: explain analyze is your friend

Reply via email to