ok.... I&#39;ll submit the bug report and attach/include my &quot;findings/solution&quot;. In the mean time I hope that either I can get the time to do this, or bribe someone else at the company to do it :)<br>Cheers<br>Ati.<br>
<br><br><div class="gmail_quote">On Feb 17, 2008 8:19 PM, Julian Hyde &lt;<a href="mailto:jhyde@pentaho.org">jhyde@pentaho.org</a>&gt; wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">




<div>
<div dir="ltr" align="left"><span><font color="#000080" face="Verdana" size="2">I don&#39;t think any progress has been made on this issue. I 
looked at the history of the email thread, and it quickly diverged into another 
issue. I ran the tests, but as I said, I need that regression test, and I&#39;m not 
going to have any time soon to write it. Maybe someone else will write it - my 
description re. TestAggregationManager is as much as I&#39;ll give as far as 
&#39;pointers&#39; - any more and I&#39;d be writing the code myself - but I have a feeling 
that no one else will write the test.</font></span></div>
<div dir="ltr" align="left"><span><font color="#000080" face="Verdana" size="2"></font></span>&nbsp;</div>
<div dir="ltr" align="left"><span><font color="#000080" face="Verdana" size="2">So, please log a bug with all of this information in it, so 
it doesn&#39;t get forgotten, and we will get to it one day.</font></span></div>
<div dir="ltr" align="left"><span><font color="#000080" face="Verdana" size="2"></font></span>&nbsp;</div>
<div dir="ltr" align="left"><span><font color="#000080" face="Verdana" size="2">Julian</font></span></div><br>
<blockquote style="border-left: 2px solid rgb(0, 0, 128); padding-left: 5px; margin-left: 5px; margin-right: 0px;">
  <div dir="ltr" align="left" lang="en-us">
  <hr>
  <font face="Tahoma" size="2"><div class="Ih2E3d"><b>From:</b> <a href="mailto:mondrian-bounces@pentaho.org" target="_blank">mondrian-bounces@pentaho.org</a> 
  [mailto:<a href="mailto:mondrian-bounces@pentaho.org" target="_blank">mondrian-bounces@pentaho.org</a>] <b>On Behalf Of </b>Ati 
  Rosselet<br></div><b>Sent:</b> Sunday, February 17, 2008 5:04 AM<div><div></div><div class="Wj3C7c"><br><b>To:</b> 
  Mondrian developer mailing list<br><b>Subject:</b> Re: [Mondrian] Problem with 
  aggregate tables..<br></div></div></font><br></div><div><div></div><div class="Wj3C7c">
  <div></div>Julian (and others?)<br>Ok... I give up.. I have no idea how to 
  properly test these changes.&nbsp; Are they still applicable to the current 
  CVS version? (I assume so.. since I haven&#39;t seen any notes/commits pertaining 
  to this problem).&nbsp;&nbsp; You mentioned that you were going to run it 
  through the regression suite to see if any problems popped up - but I never 
  heard of any results.. Does this mean that basically all is ok?&nbsp; This fix 
  is kind of important.. pretty vital to us in fact, and I&#39;m surprised no-one 
  else has encountered the problem I address in it.&nbsp;&nbsp; If you (or 
  anyone) would be willing to either write the test, or give me some pointers 
  (I&#39;m completely lost - probably partly from being in the middle of a large 
  .NET project :)), I&#39;d be very grateful.&nbsp;&nbsp; I&#39;ll try to get the CVS 
  version of 3.0 and see if the patches can still be 
  applied.<br><br>Cheers<br>Ati<br><br>
  <div class="gmail_quote">2007/10/23 Julian Hyde &lt;<a href="mailto:julianhyde@speakeasy.net" target="_blank">julianhyde@speakeasy.net</a>&gt;:<br>
  <blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
    <div>
    <div dir="ltr" align="left"><span><font color="#000080" face="Verdana" size="2">Ati,</font></span></div>
    <div dir="ltr" align="left"><span><font color="#000080" face="Verdana" size="2"></font></span>&nbsp;</div>
    <div dir="ltr" align="left"><span><font color="#000080" face="Verdana" size="2">Thanks 
    for the fix. I&#39;ve run your changes through the regression suite. Looking 
    good so far. I&#39;ll run the suite in all of its various property settings 
    tonight and see if anything pops out.</font></span></div>
    <div dir="ltr" align="left"><span><font color="#000080" face="Verdana" size="2"></font></span>&nbsp;</div>
    <div dir="ltr" align="left"><span><font color="#000080" face="Verdana" size="2">Since 
    this change is a bug fix, I need a regression test before I will check in. I 
    think you could construct a testcase based on the foodmart database. Add the 
    test to TestAggregationManager; testMultipleMeasures&nbsp;is an example of a 
    test which runs an MDX query and checks that a particular SQL query is 
    generated. You will also need to use TestContext.create() to alter the XML; 
    testKeyExpressionCardinalityCache is an example of a test which does 
    that.</font></span></div>
    <div dir="ltr" align="left"><span><font color="#000080" face="Verdana" size="2"></font></span>&nbsp;</div>
    <div dir="ltr" align="left"><span><font color="#000080" face="Verdana" size="2">Julian</font></span></div><br>
    <blockquote style="border-left: 2px solid rgb(0, 0, 128); padding-left: 5px; margin-left: 5px; margin-right: 0px;">
      <div dir="ltr" align="left" lang="en-us">
      <hr>
      <font face="Tahoma" size="2">
      <div><b>From:</b> <a href="mailto:mondrian-bounces@pentaho.org" target="_blank">mondrian-bounces@pentaho.org</a> [mailto:<a href="mailto:mondrian-bounces@pentaho.org" target="_blank">mondrian-bounces@pentaho.org</a>] <b>On Behalf Of </b>Ati 
      Rosselet<br></div><b>Sent:</b> Monday, October 22, 2007 5:23 
      PM<br><b>To:</b> Mondrian developer mailing list<br><b>Subject:</b> Re: 
      [Mondrian] Problem with aggregate tables..<br></font><br></div>
      <div>
      <div></div>
      <div>
      <div></div>sorry... cut and paste got me.. end of first function ended up 
      after the end of the 2nd modified function. 
      so<br><br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      /**<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; * Finds ALL the 
      child tables of the fact table with the given 
      columnName<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; * used in 
      their left join condition. This is used by the AggTableManager 
      <br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; * while 
      characterizing the fact table columns. (previous only returned FIRST 
      child)<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      */<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; public 
      ArrayList&lt;RolapStar.Table&gt; findTablesWithLeftJoinCondition(final 
      String columnName) <br>
      <div>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      {<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      java.util.ArrayList&lt;RolapStar.Table&gt; tables = new 
      java.util.ArrayList&lt;RolapStar.Table&gt;();<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      for (Table child : getChildren()) 
      {<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      Condition condition = 
      child.joinCondition;<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      if (condition != null) 
      {<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      if (condition.left instanceof MondrianDef.Column) 
      {<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      MondrianDef.Column mcolumn 
      =<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      ( MondrianDef.Column) 
      condition.left;<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      if (mcolumn.name.equals(columnName)) 
      {<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      tables.add(child);<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      }<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      }<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      } </div>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      }<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      return 
      (tables.size()==0?null:tables);<br>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; 
      }<br><br>and&nbsp; remove the return part from other 
      function..<br><br>Cheers..<br>Ati<br></div></div></blockquote></div><br>_______________________________________________<br>Mondrian 
    mailing list<br><a href="mailto:Mondrian@pentaho.org" target="_blank">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>
<br></blockquote></div><br></div></div></blockquote></div>
<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>
<br></blockquote></div><br>