Dave,

  Just a few comments regarding your pg_dump lock timeout patch (in
  general I like the concept and agree with adding it):

  - No validity checking that the argument passed in has anything to do
	with a number.  The backend will do this, but it strikes me as a bit
	odd to not do any checking at argument processing time.

  - You call the argument 'wait time' in the documentation, but 'DELAY'
	in the command-line help.  I'd recommend using one term and sticking
	to it.  You're already two lines in the command-line help, you can
	spell it out as 'WAIT_TIME' or similar.

  - getTables() uses different variables for each query, and I'm
	inclined to agree with that approach to make following the code
	easier.  I'd encourage you to add a new variable for the
	statement_timeout query rather than reusing the lockqry variable.
	You could even offset this by removing the unused delqry variable.

  Otherwise, looks good to me.

  	Thanks,

		Stephen

In response to

• pg_dump lock timeout at 2008-05-11 11:30:47 from daveg

pgsql-hackers by date

Next:	From: Yoshiyuki Asaba	Date: 2008-07-03 02:16:49
	Subject: Re: Git Repository for WITH RECURSIVE and others
Previous:	From: Tom Lane	Date: 2008-07-03 00:51:59
	Subject: Re: Location for pgstat.stat

pgsql-patches by date

Next:	From: Tom Lane	Date: 2008-07-03 03:38:29
	Subject: Re: Patch to change psql default banner v6
Previous:	From: David Fetter	Date: 2008-07-02 23:11:01
	Subject: WITH RECURSIVE updated to CVS TIP