[Mondrian] Update on retroweaver 2.0.7 upgrade
Eric McDermid
mcdermid at stonecreek.com
Tue Feb 3 15:53:02 EST 2009
On Feb 3, 2009, at 1:44 PM, Julian Hyde wrote:
>
>
> It has to be said: RowsetDefinition is a beast of a class. It is an
> enum
> 7000 lines long with about 20 members each of which has its own method
> implementations and is therefore effectivly an inner class, and
> another 24
> inner classes. I don't apologise for that - I think keeping it all
> in one
> file helps to keep things uniform - but I can see how retroweaver
> might have
> problems. Point out this fact to the Retroweaver developers if they
> have not
> noticed!
I'm sure they will if they haven't already. It's pretty hard to miss.
I'm inclined to give Xavier a shot at diagnosing the problem before
going back and restructuring RowsetDefinition. The earlier version of
retroweaver seems to have handled RowsetDefinition just fine, so this
may just be a corner case that's accidentally been broken and not yet
covered in the test suite.
It's also still possible that this is just another environmental
problem on my end with the setup of the weaving phase. If that's the
case, I should know soon.
-- Eric
More information about the Mondrian
mailing list