On Sat, Jul 17, 2010 at 3:59 AM, Alex Hunsaker <badalex@gmail.com> wrote:
> Patch looks and tests good to me. Only thing that seemed to be
> missing was documentation of the new pg_dump(all) and guc params.
> Find attached a stab at this. Yeah the docs I added need work, but I
> figure if you are anything like me its easier to work/tweak from a
> (crappy) base... At least with docs :). I also bumped the version
> check to 90100 from 90000.
Thanks for the review. Committed with some further cleanup.
Unfortunately, as I noticed just after committing, you only fixed ONE
of the two references to 90000, so I have corrected that with a second
commit.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise Postgres Company