Re: [PATCHES] pg_dump --no-tablespaces patch
"Gavin M. Roy" <[EMAIL PROTECTED]> 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 -- Sent via pgsql-patches mailing list (pgsql-patches@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-patches
Re: [PATCHES] 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 <[EMAIL PROTECTED]> 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
Re: [PATCHES] 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 <[EMAIL PROTECTED]> 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
Re: [PATCHES] 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
Re: [PATCHES] 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 <[EMAIL PROTECTED]> 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
Re: [PATCHES] 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
Re: [PATCHES] 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-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 <[EMAIL PROTECTED]> 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
Re: [PATCHES] 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 <[EMAIL PROTECTED]> 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 <[EMAIL PROTECTED]> 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
Re: [PATCHES] 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 <[EMAIL PROTECTED]> 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
Re: [PATCHES] pg_dump --no-tablespaces patch
Yes, Tom suggested 8.4 would be a likely candidate should it be accepted. On 7/15/07, Dave Page <[EMAIL PROTECTED]> wrote: 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.
Re: [PATCHES] 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. ---(end of broadcast)--- TIP 3: Have you checked our extensive FAQ? http://www.postgresql.org/docs/faq
[PATCHES] pg_dump --no-tablespaces patch
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 pg_dump-8.2.4-no-tablespace-option.patch Description: Binary data ---(end of broadcast)--- TIP 2: Don't 'kill -9' the postmaster