Re: directory archive format for pg_dump
Joachim Wieland <joe@mcknight.de>
From: Joachim Wieland <joe@mcknight.de>
To: Heikki Linnakangas <heikki.linnakangas@enterprisedb.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, José Arthur Benetasso Villanova <jose.arthur@gmail.com>, pgsql-hackers@postgresql.org
Date: 2010-11-29T05:11:46Z
Lists: pgsql-hackers
Attachments
- pg_dump-compression-refactor.diff (text/x-patch) patch
- pg_dump-directory.diff (text/x-patch) patch
On Mon, Nov 22, 2010 at 3:44 PM, Heikki Linnakangas <heikki.linnakangas@enterprisedb.com> wrote: > * wrap long lines > * use extern in function prototypes in header files > * "inline" some functions like _StartDataCompressor, _EndDataCompressor, > _DoInflate/_DoDeflate that aren't doing anything but call some other > function. So here is a new round of patches. It turned out that the feature to allow to also restore files from a different dump and with a different compression required some changes in the compressor API. And in the end I didn't like all the #ifdefs either and made a less #ifdef-rich version using function pointers. The downside now is that I have created quite a few one-line functions that Heikki doesn't like all that much, but I assume that they are okay in this case on the grounds that the public compressor interface is calling the private implementation of a certain compressor. Joachim