Simon Riggs wrote:
> On Mon, 2005-09-19 at 13:21 -0400, Bruce Momjian wrote:
> 
> > OK.  I tested it and it actually works
> 
> Gosh, that says a lot about my code quality. I will strive to improve
> from "actually works" to "works as expected" for future patches.
> 
> > and I added documentation
> > suggesting its usage. 
> 
> > !           If <varname>log_min_error_statement</> is set to
> > !         <literal>ERROR</> or lower, the statement that timed out will 
> > also be
> > 
> 
> Might I suggest "Even if" rather than "if"?

I don't see why "even if" makes sense here.  There is no expectation
that log_min_error_statement wouldn't work as documented.  I am more
suggesting that it is a nifity way to use the option.

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

---------------------------(end of broadcast)---------------------------
TIP 5: don't forget to increase your free space map settings

Reply via email to