Hi Julian,<br><br>Thanks for the review.<br><br>This checkin is meant to be a first cut before I have to change machines. As of now it still sits in the private branch. I will make sure when the branch gets integrated, all the issues are addressed.<br>
<br>Rushan<br><br><div class="gmail_quote">On Fri, Jun 19, 2009 at 9:16 AM, Julian Hyde <span dir="ltr">&lt;<a href="mailto:jhyde@pentaho.com">jhyde@pentaho.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Very nice.<br>
<br>
Can you add a test case for this? Using a few crossjoins, it should be easy<br>
to construct an example query that takes (say) 2 minutes without your change<br>
and 2 seconds after. If the functionality regressed, we would probably<br>
notice a 2 minute increase in the length of the test run.<br>
<br>
Btw, there are tabs all over your source code. I&#39;m guessing you didn&#39;t run<br>
checkFile. :)<br>
<br>
Julian<br>
<br>
&gt; From: Rushan Chen [mailto:<a href="mailto:rchen@lucidera.com">rchen@lucidera.com</a>]<br>
&gt; Subject: Eigenbase perforce change 12884 for review<br>
&gt;<br>
&gt; <a href="http://p4web.eigenbase.org/@md=d&amp;c=6PU@//12884?ac=10" target="_blank">http://p4web.eigenbase.org/@md=d&amp;c=6PU@//12884?ac=10</a><br>
&gt;<br>
&gt; Change 12884 by rchen@rchen.marmot.eigenbase on 2009/06/18 16:30:59<br>
&gt;<br>
&gt;       MONDRIAN: improve the performance of Rank function accepting 3<br>
&gt;       args. First cut of implemetation.<br>
<br>
<br>
_______________________________________________<br>
Mondrian mailing list<br>
<a href="mailto:Mondrian@pentaho.org">Mondrian@pentaho.org</a><br>
<a href="http://lists.pentaho.org/mailman/listinfo/mondrian" target="_blank">http://lists.pentaho.org/mailman/listinfo/mondrian</a><br>
</blockquote></div><br>