On 26 February 2015 at 09:27, Dean Rasheed <dean.a.rasheed@gmail.com> wrote:
> I think this should probably be committed as 2 separate patches...
On closer inspection, the 2 parts are interrelated since the new
self-join test that tests the inheritance planner changes also
requires the rewriter changes, otherwise it falls over with an
infinite recursion error in the rewriter. I suppose the rewriter
changes could be committed first with the self-join without
inheritance test to test the fix to the infinite recursion problem,
but I think it's probably easier to just treat this as a single set of
related bug fixes.
Attached is an updated patch. There are no more code changes, but I've
added a few more regression tests.
Regards,
Dean