pg_dump --no-tablespaces patch

Gavin M. Roy wrote:

This is the patch I proposed on hackers to make pg_dump optionally
ignore tablespaces. The patch is against 8.2.4. If I should be
applying it to CVS head or what not, please let me know (along with any
thoughts, concerns or issues).

Hi Gavin,

It should be against -head - there are changes to pg_dump in 8.3 that
may well prevent a patch against 8.2.4 applying cleanly.

As it's you, I assume you realise that you're too late for 8.3?

Regards, Dave.

Yes, Tom suggested 8.4 would be a likely candidate should it be accepted.

This has been saved for the 8.4 release:

http://momjian.postgresql.org/cgi-bin/pgpatches_hold

---------------------------------------------------------------------------

Gavin M. Roy wrote:

This is the patch I proposed on hackers to make pg_dump optionally ignore
tablespaces. The patch is against 8.2.4. If I should be applying it to CVS
head or what not, please let me know (along with any thoughts, concerns or
issues).

Gavin

[ Attachment, skipping... ]

---------------------------(end of broadcast)---------------------------
TIP 2: Don't 'kill -9' the postmaster

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

I wanted to push to see if I made a version of this patch against CVS
head if we could see it included in 8.3.

It's of clear value for people who use pg_dump and tablespaces. HOT
and tsearch development still continues and implementations of each
feature are still being discussed/debated as of this morning. It
seems to me that 8.3 code base is not static and we're not at the bug
cleanup phase of nearing release.

This patch is minor and makes admin, backup and recovery much easier
for people who use the tablespace feature and elect not to use
pg_dumpall. I think waiting for 8.4 at this point is a bit silly,
given the scope of code and impact of the patch.

Gavin

Gavin M. Roy wrote:

I wanted to push to see if I made a version of this patch against CVS
head if we could see it included in 8.3.

It's of clear value for people who use pg_dump and tablespaces. HOT
and tsearch development still continues and implementations of each
feature are still being discussed/debated as of this morning. It
seems to me that 8.3 code base is not static and we're not at the bug
cleanup phase of nearing release.

This patch is minor and makes admin, backup and recovery much easier
for people who use the tablespace feature and elect not to use
pg_dumpall. I think waiting for 8.4 at this point is a bit silly,
given the scope of code and impact of the patch.

You already got an answer to this question, and you replied:

http://archives.postgresql.org/pgsql-patches/2007-07/msg00167.php

If you want it in 8.3 you are going to need some community members to
take ownership of the patch, take responsibility for it, testing it, etc,
and argue that a feature addition that arrived in July should be added.
Not impossible, but you are going to need advocates for this.

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

Bruce Momjian wrote:

If you want it in 8.3 you are going to need some community members to
take ownership of the patch, take responsibility for it, testing it, etc,
and argue that a feature addition that arrived in July should be added.
Not impossible, but you are going to need advocates for this.

At this stage we should be kicking things out of the list, not adding to
it, IMNSHO. It really doesn't look to me like HOT is ready, from what I
can see of the discussion, and desirable as it is I think it should
probably wait.

Just my USD0.02 worth.

cheers

andrew

Andrew Dunstan wrote:

Bruce Momjian wrote:

If you want it in 8.3 you are going to need some community members to
take ownership of the patch, take responsibility for it, testing it, etc,
and argue that a feature addition that arrived in July should be added.
Not impossible, but you are going to need advocates for this.

At this stage we should be kicking things out of the list, not adding to
it, IMNSHO. It really doesn't look to me like HOT is ready, from what I
can see of the discussion, and desirable as it is I think it should
probably wait.

Are you saying HOT should wait or --no-tablespaces?

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

Bruce Momjian wrote:

Andrew Dunstan wrote:

Bruce Momjian wrote:

If you want it in 8.3 you are going to need some community members to
take ownership of the patch, take responsibility for it, testing it, etc,
and argue that a feature addition that arrived in July should be added.
Not impossible, but you are going to need advocates for this.

At this stage we should be kicking things out of the list, not adding to
it, IMNSHO. It really doesn't look to me like HOT is ready, from what I
can see of the discussion, and desirable as it is I think it should
probably wait.

Are you saying HOT should wait or --no-tablespaces?

--no-tablespaces definitely, HOT too unless there is a short clear path
to committal (which seems unlikely given the tenor of the discussions).

cheers

andrew

Andrew Dunstan wrote:

At this stage we should be kicking things out of the list, not adding to
it, IMNSHO. It really doesn't look to me like HOT is ready, from what I
can see of the discussion, and desirable as it is I think it should
probably wait.

Are you saying HOT should wait or --no-tablespaces?

--no-tablespaces definitely,

Agreed.

HOT too unless there is a short clear path
to committal (which seems unlikely given the tenor of the discussions).

You can't expect something like HOT to hit the mark 100% on its first
review. If we expected 100%, nothing complex would get applied. We
have a few weeks for HOT to mature now that it has feedback from Tom.

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Bruce Momjian wrote:

Andrew Dunstan wrote:

At this stage we should be kicking things out of the list, not adding to

HOT too unless there is a short clear path
to committal (which seems unlikely given the tenor of the discussions).

You can't expect something like HOT to hit the mark 100% on its first
review.

Aren't we on review 14?

Joshua D. Drake

- --

=== The PostgreSQL Company: Command Prompt, Inc. ===
Sales/Support: +1.503.667.4564 24x7/Emergency: +1.800.492.2240
PostgreSQL solutions since 1997 http://www.commandprompt.com/
UNIQUE NOT NULL
Donate to the PostgreSQL Project: http://www.postgresql.org/about/donate
PostgreSQL Replication: http://www.commandprompt.com/products/

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFG1vhMATb/zqfZUUQRAqGcAJ0faHAM+9E0hlZzTcdyg5IMvSEifACgrRw5
KbOA5FN1oj+dEUl9AbZ2lzc=
=1lHk
-----END PGP SIGNATURE-----

Joshua D. Drake wrote:

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Bruce Momjian wrote:

Andrew Dunstan wrote:

At this stage we should be kicking things out of the list, not adding to

HOT too unless there is a short clear path
to committal (which seems unlikely given the tenor of the discussions).

You can't expect something like HOT to hit the mark 100% on its first
review.

Aren't we on review 14?

We are on versoin 14 that they generated with Heikki's feedback and
others, but no Tom's.

--
Bruce Momjian <bruce@momjian.us> http://momjian.us
EnterpriseDB http://www.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

"Gavin M. Roy" <gmr@myyearbook.com> writes:

This is the patch I proposed on hackers to make pg_dump optionally ignore
tablespaces. The patch is against 8.2.4. If I should be applying it to CVS
head or what not, please let me know (along with any thoughts, concerns or
issues).

Applied with minor fixes and addition of documentation.

regards, tom lane