PDA

View Full Version : RE: Re : [Mondrian]



Pappyn Bart
01-22-2007, 07:40 AM
We must get one thing straight : regressions tests - except for paralleltest - are using
only one thread. All other tests are not multi threaded.
________________________________

From: mondrian-bounces (AT) pentaho (DOT) org [mailto:mondrian-bounces (AT) pentaho (DOT) org] On Behalf Of michael bienstein
Sent: maandag 22 januari 2007 12:28
To: Mondrian developer mailing list
Subject: Re : [Mondrian] Re:VirtualCubeTest.testCalculatedMemberAcrossCubesfailing on SMP


I think a debugger won't work because debugging multi-threaded apps is really really difficult.

I did a code review but couldn't work out where it came from. I assume the thread ID is positive and unique so that if the same Thread object is attached to an underlying os thread it won't be doing the same thing, otherwise the == operator would fail. The only thing I'd do differently is wrap the try {} finally bloc in the RolapResult around more than just the evaluateBody() method because the axes get evaluated out of it. But the bug doesn't come from there.

Oh and you might get conflicting data because in one MDX query we can open and use multiple JDBC Connections. Should use a ThreadLocal to keep the open connection and re-use it.

Michael


----- Message d'origine ----
De : Pappyn Bart <Bart.Pappyn (AT) vandewiele (DOT) com>