On Tue, Sep 15, 2009 at 09:53:11PM -0400, Stephen Frost wrote:
> Michael,
>
> I just wanted to follow-up on your pgbench patch. The latest version
> that I see is from August 13th. Is that the correct patch to be
> reviewing? Do you have any other updates on it?
>
> Thanks!
>
> Stephen
Hi!
Some comments about this patch:
- You have DOS-style carriage returns, which interfere with the patch application on Unix systems.
- We'd like to see return specifically return a value (lines 1008 and 1022 in the patched version)
- We'd like to see something done with the return value from system (line 1026 in patched version)
- pg_bench functions as expected, however, your example script given on the wiki page for this patch fails
(http://wiki.postgresql.org/wiki/Pgbench:_shell_command). Can we have an example that works so we can check it out?
It'snot really clear to us how this will be useful to others.
Thanks!
Gabrielle & Dan
