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

From: "Qingqing Zhou" <zhouqq(at)cs(dot)toronto(dot)edu>
To: pgsql-committers(at)postgresql(dot)org
Subject: Re: pgsql: Have test_fsync honor -f filename argument.
Date: 2005-11-16 01:53:29
Message-ID: dle3dv$1g9r$1@news.hub.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers


"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
---

Regards,
Qingqing

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Bruce Momjian 2005-11-16 03:32:06 pgsql: Update error message and documentation for fsync test.
Previous Message Bruce Momjian 2005-11-16 01:34:11 pgsql: Update test_fsync to honor -f.