Re: [PATCH] Fix pg_dump --no-tablespaces for the custom format - Mailing list pgsql-hackers

From Tom Lane
Subject Re: [PATCH] Fix pg_dump --no-tablespaces for the custom format
Date
Msg-id 20336.1589588201@sss.pgh.pa.us
Whole thread Raw
In response to [PATCH] Fix pg_dump --no-tablespaces for the custom format  (Christopher Baines <mail@cbaines.net>)
Responses Re: [PATCH] Fix pg_dump --no-tablespaces for the custom format
List pgsql-hackers
Christopher Baines <mail@cbaines.net> writes:
> So I'm new to poking around in the PostgreSQL code, so this is a bit of
> a shot in the dark. I'm having some problems with pg_dump, and a
> database with tablespaces. A couple of the tables are not in the default
> tablespace, and I want to ignore this for the dump.

I think you've misunderstood how the pieces fit together.  A lot of
the detail-filtering switches, including --no-tablespaces, work on
the output side of the "archive" format.  While you can't really tell
the difference in pg_dump text mode, the implication for custom-format
output is that the info is always there in the archive file, and you
give the switch to pg_restore if you don't want to see the info.
This is more flexible since you aren't compelled to make the decision
up-front, and it doesn't really cost anything to include such info in
the archive.  (Obviously, table-filtering switches don't work that
way, since with those there can be a really large cost in file size
to include unwanted data.)

So from my perspective, things are working fine and this patch would
break it.

If you actually want to suppress this info from getting into the
archive file, you'd have to give a very compelling reason for
breaking this behavior for other people.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: Potentially misleading name of libpq pass phrase hook
Next
From: Tom Lane
Date:
Subject: Re: Potentially misleading name of libpq pass phrase hook