| From: | Euler Taveira de Oliveira <euler(at)timbira(dot)com> |
|---|---|
| To: | Simon Riggs <simon(at)2ndquadrant(dot)com> |
| Cc: | pgsql-patches(at)postgresql(dot)org |
| Subject: | Re: xlog directory at initdb time |
| Date: | 2006-12-28 22:40:52 |
| Message-ID: | 20061228224052.GA3041@timbira.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers pgsql-patches |
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/
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Simon Riggs | 2006-12-28 22:47:23 | Re: pg_standby |
| Previous Message | Andrew Dunstan | 2006-12-28 22:16:58 | Re: TODO: GNU TLS |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Simon Riggs | 2006-12-28 22:47:23 | Re: pg_standby |
| Previous Message | Alvaro Herrera | 2006-12-28 22:15:25 | Re: Recent SIGSEGV failures in buildfarm HEAD |