Re: [HACKERS] Optimizer cleanup to avoid redundant work on joins

> Er ... um ... ahem ... DUH!  The reason 6.5.3 works is that it does in
> fact keep join restrictinfo pointers in JoinPaths.  I had eliminated
> those pointers (the thoroughly undocumented "pathinfo" field) because
> I thought that the lists were always the same as the parent relations'
> restrictinfo lists.  Which they were --- at the time of creation of a
> JoinPath.  What I missed was that prune.c moved a joinpath to belong
> to a different RelOptInfo with (potentially) a different restrictinfo
> list, but the joinpath needs to keep its original restrictinfo list.
> 
> In other words, I broke it.
> 
> Since surgery needs to be done anyway, I'm inclined to press ahead
> with the changes I was going to put off.  On the other hand, if the
> patient had a vote, it might ask for a second opinion ;-)

Go for it.  Beta is for testing.  No better time to break things than
the present.

This is the first time I remember hearing about pre-beta release jitters
from many people.

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