Unix Technical Forum

Re: pg_dump --no-tablespaces patch

This is a discussion on Re: pg_dump --no-tablespaces patch within the Pgsql Patches forums, part of the PostgreSQL category; --> 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 ...


Go Back   Unix Technical Forum > Database Server Software > PostgreSQL > Pgsql Patches

Register FAQ Members List Calendar Search Today's Posts Mark Forums Read
  #1 (permalink)  
Old 04-18-2008, 10:24 AM
Bruce Momjian
 
Posts: n/a
Default Re: pg_dump --no-tablespaces patch


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

---------------------------(end of broadcast)---------------------------
TIP 3: Have you checked our extensive FAQ?

http://www.postgresql.org/docs/faq

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #2 (permalink)  
Old 04-18-2008, 10:32 AM
Gavin M. Roy
 
Posts: n/a
Default Re: pg_dump --no-tablespaces patch

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

On 7/17/07, Bruce Momjian <bruce@momjian.us> wrote:
>
> 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. +
>


---------------------------(end of broadcast)---------------------------
TIP 7: You can help support the PostgreSQL project by donating at

http://www.postgresql.org/about/donate

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #3 (permalink)  
Old 04-18-2008, 10:32 AM
Bruce Momjian
 
Posts: n/a
Default Re: pg_dump --no-tablespaces patch

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...7/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. +

---------------------------(end of broadcast)---------------------------
TIP 3: Have you checked our extensive FAQ?

http://www.postgresql.org/docs/faq

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #4 (permalink)  
Old 04-18-2008, 10:32 AM
Andrew Dunstan
 
Posts: n/a
Default Re: pg_dump --no-tablespaces patch



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

---------------------------(end of broadcast)---------------------------
TIP 4: Have you searched our list archives?

http://archives.postgresql.org

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #5 (permalink)  
Old 04-18-2008, 10:32 AM
Bruce Momjian
 
Posts: n/a
Default Re: pg_dump --no-tablespaces patch

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

---------------------------(end of broadcast)---------------------------
TIP 4: Have you searched our list archives?

http://archives.postgresql.org

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #6 (permalink)  
Old 04-18-2008, 10:32 AM
Andrew Dunstan
 
Posts: n/a
Default Re: pg_dump --no-tablespaces patch



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

---------------------------(end of broadcast)---------------------------
TIP 7: You can help support the PostgreSQL project by donating at

http://www.postgresql.org/about/donate

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #7 (permalink)  
Old 04-18-2008, 10:32 AM
Bruce Momjian
 
Posts: n/a
Default Re: pg_dump --no-tablespaces patch

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

---------------------------(end of broadcast)---------------------------
TIP 7: You can help support the PostgreSQL project by donating at

http://www.postgresql.org/about/donate

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #8 (permalink)  
Old 04-18-2008, 10:32 AM
Joshua D. Drake
 
Posts: n/a
Default Re: pg_dump --no-tablespaces patch

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

---------------------------(end of broadcast)---------------------------
TIP 6: explain analyze is your friend

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
  #9 (permalink)  
Old 04-18-2008, 10:32 AM
Bruce Momjian
 
Posts: n/a
Default Re: pg_dump --no-tablespaces patch

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

---------------------------(end of broadcast)---------------------------
TIP 6: explain analyze is your friend

Digg this Post!Add Post to del.icio.usBookmark Post in TechnoratiFurl this Post!
Reply With Quote
Reply


Thread Tools
Display Modes

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Trackbacks are On
Pingbacks are On
Refbacks are On
Forum Jump


All times are GMT. The time now is 10:15 PM.


Powered by vBulletin® Version 3.6.5
Copyright ©2000 - 2008, Jelsoft Enterprises Ltd.
SEO by vBSEO 3.2.0
www.UnixAdminTalk.com