I can't wait to apply this...

> 
> I'm resubmitting this patch from a while ago, now that 7.0 is out. If
> you cast your minds back, this patch allows update and delete to work on
> inheritance hierarchies just like it now works on select. It also uses
> the Informix/Illustra model for subclasses - i.e. "ONLY", as was
> discussed at length before.
> 
> Please point out anything I've screwed up so I can post a final version.
> In particular I forgot where you change the initdb db version thingy,
> but I don't want to do that anyway till everything else is correct.

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

Wed, 17 May 2000 12:01:55 -0400 (EDT)
Bruce Momjian <pgman@candle.pha.pa.us>
Re: OO Patch
Chris writes:

> I'm resubmitting this patch from a while ago, now that 7.0 is out.

I don't recall that discussion ever ending with a definite design
document. Could you at least point out which of the many proposals this
refers to? What I do remember in fact is that the core decided that the
various proposals need to be checked against SQL3 before anything can
happen and as far as I can see you definitely didn't do that.

> If you cast your minds back, this patch allows update and delete to
> work on inheritance hierarchies just like it now works on select.

I don't think that's a good idea. If I have an inheritance hierarchy A < B
< C and I update a row in A "only" then I break the hierarchy. (I'm also
wondering how you would do that, since you would have to make a copy of
the row for A and then keep the old copies around for B and C.) SQL3
violation right there. That also goes for the various ALTER TABLE [ONLY]
syntax additions. If I add a row to A only then B is no longer a subtable
of A. One thing I see you didn't change is the CREATE TABLE syntax,
although that could posibly have used it.

> It also uses the Informix/Illustra model for subclasses - i.e. "ONLY",
> as was discussed at length before.

SELECT ONLY is cool, that's the SQL3 way.


All in all I think it's great that you're tackling this but this patch
looks very suspect to me. Rolling your own inheritance model when there's
decades of scientific research out there that some presumably smart guys
wrote down in a (now official) standards document doesn't seem like a wise
thing to do.


PS: Could you elaborate on that FAQ_DEV change?

-- 
Peter Eisentraut                  Sernanders vï¿½g 10:115
peter_e@gmx.net                   75262 Uppsala
http://yi.org/peter-e/            Sweden


Thu, 18 May 2000 00:04:31 +0200 (CEST)
Peter Eisentraut <peter_e@gmx.net>
Re: OO Patch
I'm resubmitting this patch from a while ago, now that 7.0 is out. If
you cast your minds back, this patch allows update and delete to work on
inheritance hierarchies just like it now works on select. It also uses
the Informix/Illustra model for subclasses - i.e. "ONLY", as was
discussed at length before.

Please point out anything I've screwed up so I can post a final version.
In particular I forgot where you change the initdb db version thingy,
but I don't want to do that anyway till everything else is correct.
Thu, 18 May 2000 11:12:26 +1000
Chris <chris@bitmead.com>
OO Patch