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
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 |