AW: AW: AW: Modified pg_dump & new pg_restore need test ing...

Started by Zeugswetter Andreas SBover 25 years ago6 messages
#1Zeugswetter Andreas SB
ZeugswetterA@wien.spardat.at

Philip Warner [mailto:pjw@rhyme.com.au] wrote:

At 10:39 3/07/00 +0200, Zeugswetter Andreas SB wrote:

Could the text format be changed in a compatible way,
that would allow pg_restore's features ? I am thinking of

inserted comment

lines that describe the needed sections.

Quite possibly; although part of my motivation in writing to
the new format
was to avoid writing a text parser.

Yes, you are probably right.

There is no reason why it couldn't use
a text file by doing a pass through the file to construct a
TOC, but this
seems like a bad idea for large backups, and not really
necessary for small
backups (especially with the -Z option).

That said, I'm happy to give it a go - the output file is
already pretty
'parsable'.

I guess it is probably really not worth the trouble.

Imho the new default format does not need to be compatible
with pg_dump's output. I would still not compress by default.

I'm happy to do this; at the moment it uses zlib default
compression by

Do we already check for zlib ? It is probably not available everywhere.

Andreas

#2Philip Warner
pjw@rhyme.com.au
In reply to: Zeugswetter Andreas SB (#1)
Re: AW: AW: AW: Modified pg_dump & new pg_restore need testing...

At 10:56 3/07/00 +0200, Zeugswetter Andreas SB wrote:

Do we already check for zlib ? It is probably not available everywhere.

That may be true - although I thought it was pretty universal.

I will have a look at using something 'HAVE_ZLIB' setting to enable/disable
compressed output...I presume 'configure' needs to be changed? Is that right?

----------------------------------------------------------------
Philip Warner | __---_____
Albatross Consulting Pty. Ltd. |----/ - \
(A.C.N. 008 659 498) | /(@) ______---_
Tel: (+61) 0500 83 82 81 | _________ \
Fax: (+61) 0500 83 82 82 | ___________ |
Http://www.rhyme.com.au | / \|
| --________--
PGP key available upon request, | /
and from pgp5.ai.mit.edu:11371 |/

#3The Hermit Hacker
scrappy@hub.org
In reply to: Philip Warner (#2)
Re: AW: AW: AW: Modified pg_dump & new pg_restore need testing...

On Mon, 3 Jul 2000, Philip Warner wrote:

At 10:56 3/07/00 +0200, Zeugswetter Andreas SB wrote:

Do we already check for zlib ? It is probably not available everywhere.

That may be true - although I thought it was pretty universal.

I will have a look at using something 'HAVE_ZLIB' setting to enable/disable
compressed output...I presume 'configure' needs to be changed? Is that right?

that will be correct, but its a quick/easy change ... I'll add the code in
tonight so that its something we check for and set in config.h, so as long
as you include that, you'll have that setting available ...

#4Philip Warner
pjw@rhyme.com.au
In reply to: The Hermit Hacker (#3)
1 attachment(s)
Re: AW: AW: AW: Modified pg_dump & new pg_restore need testing...

At 22:39 3/07/00 -0300, The Hermit Hacker wrote:

that will be correct, but its a quick/easy change ... I'll add the code in
tonight so that its something we check for and set in config.h, so as long
as you include that, you'll have that setting available ...

OK. Heres a patch against CVS (pre new pg_dump). This is a complete patch,
so you can throw away the last two...

Bye for now,

Philip

Attachments:

pg_dump_cvs_120_patch.gzapplication/octet-stream; name=pg_dump_cvs_120_patch.gz; x-mac-creator=477A6970; x-mac-type=477A6970Download
#5Philip Warner
pjw@rhyme.com.au
In reply to: Philip Warner (#4)
Re: AW: AW: AW: Modified pg_dump & new pg_restore need testing...

Sorry about that; I didn't check the 'to' address before sending the patch...

----------------------------------------------------------------
Philip Warner | __---_____
Albatross Consulting Pty. Ltd. |----/ - \
(A.C.N. 008 659 498) | /(@) ______---_
Tel: (+61) 0500 83 82 81 | _________ \
Fax: (+61) 0500 83 82 82 | ___________ |
Http://www.rhyme.com.au | / \|
| --________--
PGP key available upon request, | /
and from pgp5.ai.mit.edu:11371 |/

#6Bruce Momjian
pgman@candle.pha.pa.us
In reply to: Philip Warner (#4)
Re: AW: AW: AW: Modified pg_dump & new pg_restore need testing...

Do you want this applied?

At 22:39 3/07/00 -0300, The Hermit Hacker wrote:

that will be correct, but its a quick/easy change ... I'll add the code in
tonight so that its something we check for and set in config.h, so as long
as you include that, you'll have that setting available ...

OK. Heres a patch against CVS (pre new pg_dump). This is a complete patch,
so you can throw away the last two...

Bye for now,

Philip

[ Attachment, skipping... ]

----------------------------------------------------------------
Philip Warner | __---_____
Albatross Consulting Pty. Ltd. |----/ - \
(A.C.N. 008 659 498) | /(@) ______---_
Tel: (+61) 0500 83 82 81 | _________ \
Fax: (+61) 0500 83 82 82 | ___________ |
Http://www.rhyme.com.au | / \|
| --________--
PGP key available upon request, | /
and from pgp5.ai.mit.edu:11371 |/

-- 
  Bruce Momjian                        |  http://www.op.net/~candle
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026