Re: CLUSTER TODO item

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Gavin Sherry <swm(at)linuxworld(dot)com(dot)au>
Cc: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)postgresql(dot)org
Subject: Re: CLUSTER TODO item
Date: 2002-02-22 16:15:11
Message-ID: 200202221615.g1MGFBE12866@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


Is there an updated version of this patch for 7.3?

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

Gavin Sherry wrote:
> On Thu, 11 Oct 2001, Tom Lane wrote:
>
> > Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us> writes:
> > > Can I get a status on this?
> >
> > It's not gonna happen for 7.2, I think ...
> >
> > regards, tom lane
>
> I'd love it to go out with 7.2 but I've had no time to work on this patch
> lately. The reason I need time is that, after having fiddled a fair bit,
> I've decided that there really needs to be support for the creation of a
> new relfilenode in the storage manager.
>
> The current patch works like this:
>
> Create new heap (heap_create())
> Copy tuples from old heap to new heap via index scan
> Form a new pg_class tuple
> simple_heap_update()
> update catalogue indices
> rebuild existing indices
>
> This causes an overflow in the localbuf.c so I guess this is wrong
> (included in patch on 24/sep) =). I've looked at various combinations of
> this:
>
> memcpy() the old Relation into a new Relation, update smgrunlink() the old
> Relation and heap_storage_create() the new relation. This dies because
> smgrunlink only schedules the drop, where as heap_storage_create() actually
> creates a file on the file system (open() returns with EEXIST).
>
> I've also tried just copying the structure but heap_open() relies on OID
> not relfilenode.
>
> I'm probably going about it the wrong way, but it strikes me that there
> needs to be a way to abstract the relfilenode from OID in the heap access
> code so that one can easily manipulate the heap on disk without having to
> play with OIDs.
>
> I would have included code examples/clearer description but the box I
> don't have access to the box I created the patches on atm =(.
>
> Gavin
>
>
> ---------------------------(end of broadcast)---------------------------
> TIP 4: Don't 'kill -9' the postmaster
>

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

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2002-02-22 16:17:25 Re: Automatic transactions in psql
Previous Message Tom Lane 2002-02-22 16:14:24 Re: point in time recovery and moving datafiles online