Re: pg_dump --split patch

From: Joel Jacobson <joel(at)gluefinance(dot)com>
To: Dimitri Fontaine <dimitri(at)2ndquadrant(dot)fr>
Cc: pgsql-hackers(at)postgresql(dot)org
Subject: Re: pg_dump --split patch
Date: 2010-12-30 07:13:20
Message-ID: AANLkTin7mNZQ252knDKX7F4W0TuMCNeQr8OQmGKjhSaf@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

2010/12/29 Dimitri Fontaine <dimitri(at)2ndquadrant(dot)fr>

> Please have a look at getddl:
>
> https://github.com/dimitri/getddl
>
>
Nice! Looks like a nifty tool.
When I tried it, "./getddl.py -f -F /crypt/funcs -d glue", I got the error
"No such file or directory: 'sql/schemas.sql'".

While the task of splitting objects into separate files could be solved by
an external "wrapper tool" like yours around pg_dump,
I argue it makes more sense of putting the (minimal required) logics into
pg_dump, due to a number of reasons, most importantly because it's simplier
and less complex, thus less error prone.

My patch is only a few lines of code and doesn't add any logics to pg_dump,
it merely reroutes the fwrite() system calls based on the toc entries.

Just the fact you and others had to create own tools to do the splitting
shows the feature is important, which I think should be included in the
normal pg_dump tool.

Happy new year!

Joel Jacobson
Glue Finance

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Mark Kirkwood 2010-12-30 08:55:32 Re: Vacuum of newly activated 8.3.12 standby receives warnings page xxx is uninitialized --- fixing
Previous Message Greg Smith 2010-12-30 07:02:51 Re: Re: new patch of MERGE (merge_204) & a question about duplicated ctid