Re: pgsql: Have test_fsync honor -f filename argument.

Qingqing Zhou wrote:
>
> "Bruce Momjian" <momjian(at)svr1(dot)postgresql(dot)org> wrote
> > -----------
> > Have test_fsync honor -f filename argument.
> >
>
> The patch may forget to change the error message (Also, it would be better
> to append a '\n' in die() function). Another possible problem of this test
> is that it pickups the "/var/tmp". OS may have IO optimization on temporary
> directories, which will affect the confidence of the results.
>
> Example:
> ---
> $#./test_fsync -f $HOME/test.out
> Simple write timing:
> write 4.265149
> $#./test_fsync -f /var/tmp/test.out
> Simple write timing:
> write 0.018404
> $#./test_fsync -f /tmp/test.out
> Simple write timing:
> write 0.018511
> ---

Thanks. Updated error message and added mention of memory file systems
to README.

--
Bruce Momjian | http://candle.pha.pa.us
pgman(at)candle(dot)pha(dot)pa(dot)us | (610) 359-1001
+ If your life is a hard drive, | 13 Roberts Road
+ Christ can be your backup. | Newtown Square, Pennsylvania 19073