[Mondrian] Re: VirtualCubeTest.testCalculatedMemberAcrossCubesfailing on SMP
julianhyde at speakeasy.net
Tue Jan 23 05:57:24 EST 2007
I think the problem is with how mondrian evaluates members using multiple
passes. When the measures are coming from a virtual cube, of course there
are multiple real cubes, and each of those has a cell reader. But the code
in RolapResult assumes there is only one cell reader.
Mondrian should check the cell readers for all applicable cubes, and only
emit a result when all cell readers have been populated.
I haven't implemented the fix yet, but this cause seems very plausible to
I'm not exactly sure why this problem surfaced after Bart's change - maybe
thread-local caches increased the chances of one cache being populated and
another not - or why it appears on SMP machines.
By the way, in an effort to get this working, I removed Bart's
RolapStarAggregationKey (a compound key of BitKey and thread id) and moved
to a two-tier hashing scheme. The first tier is a ThreadLocal of maps, and
the second tier is a map. Threads which want access to the global map just
skip the first tier. Given the difficulties obtaining a unique id for a
thread, using a ThreadLocal seemed cleaner. So, even though this didn't fix
the bug, I'm going to check in.
From: mondrian-bounces at pentaho.org [mailto:mondrian-bounces at pentaho.org] On
Behalf Of michael bienstein
Sent: Monday, January 22, 2007 12:06 PM
To: Mondrian developer mailing list
Subject: Re : [Mondrian] Re:
VirtualCubeTest.testCalculatedMemberAcrossCubesfailing on SMP
I've seen issues with server mode JIT before related to memory boundaries
and multiple threads. But that's mutiple threads and it was in JDK 1.4 (the
memory model changed in 1.5 I think). The issue is that the instructions in
the Java code can be run out of order to the way you've coded them. E.g.
a=1; b=2; a=b; can be run just a=2; b=2; because that's what it is
equivalent to. The only way to force it to do what you really expected is
to synchronize your accesses because that prevents the instruction
re-ordering across the memory boundary. This was an issue in Apache Struts
at one point because they used a custom Map implementation called
"FastHashMap" which gets filled with values and then flipped to be in
immutable mode. The problem was that the get() method tested if it was
flipped already without synchronizing which looked safe because the flip
flag was set only after the insertion code. But the JIT reversed the order
and the flip was done before the last insertions leading to certain problems
on high-end servers.
All that's a moot point if we can't see how multiple threads are being used.
----- Message d'origine ----
De : John V. Sichi <jsichi at gmail.com>
À : Pappyn Bart <Bart.Pappyn at vandewiele.com>
Cc : Mondrian developer mailing list <mondrian at pentaho.org>
Envoyé le : Lundi, 22 Janvier 2007, 20h10mn 24s
Objet : [Mondrian] Re: VirtualCubeTest.testCalculatedMemberAcrossCubes
failing on SMP
Something interesting: I noticed that HotSpot was automatically
selecting Server mode on the SMP machine (whereas on my laptop it
autoselects Client mode). I changed build.xml to force usage of Client
mode, and then "ant test" ran through with no failures.
Haven't tried forcing Server mode on laptop yet; if it's timing-related,
it may just be that it takes the combination of a very fast machine plus
Server mode to hit it.
Also, Bart, if you want to send me code modifications to enable tracing
targeted at debugging the problem, please do, and I'll run it through on
the SMP machine and send you the output.
John V. Sichi wrote:
> Pappyn Bart wrote:
>> Could you tell me what kind of OS is running on your
>> 4-way SMP machine ? And on you laptop ?
> SMP: Red Hat Enterprise Linux (not sure of version); JVM is HotSpot
> Laptop: Edgy Eft version of Ubuntu Linux; JVM is HotSpot 1.5.0_04-b05
> My guess is that it's likely to be a timing-sensitive thing. The
> property trigger stuff looked suspicious, but I tried disabling that out
> and it still happens. I also tried disabling
> testFormatStringExpressionCubeNoCache (since it runs just before and has
> cache disabled on a base cube underlying a virtual cube) but the failure
> still occurred.
Mondrian mailing list
Mondrian at pentaho.org
Découvrez une nouvelle façon d'obtenir des réponses à toutes vos questions !
Profitez des connaissances, des opinions et des expériences des internautes
sur Yahoo! <http://fr.rd.yahoo.com/evt=42054/*http://fr.answers.yahoo.com>
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the Mondrian