Cheers. Thanx.&nbsp;&nbsp; I&#39;d ask for cvs access, but I&#39;m not that confident of&nbsp; not breaking anything :)<br><br>I&#39;d suggest the following change to AggGen.java as well then. In both cases Iremoved - toUpperCase());<br>
650: &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; agg.getExpression(sqlQuery.getDialect().quoteIdentifier(<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;&nbsp;&nbsp; factTableName, c.getName()))); <br>892: &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; name = usage.getSymbolicName().replace(&#39; &#39;, &#39;_&#39;);&nbsp;&nbsp; // removed - toUpperCase();
<br><br>RolapAggregator.java:&nbsp; // keywords into uppercase - SQL standard and much more readable.<br><br>86:&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; return &quot;COUNT(DISTINCT &quot; + operand + &quot;)&quot;; <br><br>140: &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; buf.append(&quot;SUM(&quot;);
<br>142: &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; buf.append(&quot;) / SUM(&quot;);<br><br>170: &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; buf.append(&quot;SUM(&quot;);<br>
175: &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; buf.append(&quot;) / SUM(&quot;);<br>
<br>198:&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; buf.append(&quot;SUM(&quot;); <br><br>228: &nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; buf.append(name.toUpperCase()); // expression part in uppercase<br>231:&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; buf.append(&quot;DISTINCT &quot;);&nbsp; <br><br><br>also need to fix the expected SQL from reference SQL for 
mondrian.rolap.CacheControlTest.java<br>(CacheControlTest.ref.xml - all &quot;sum(&quot; to &quot;SUM(&quot;&nbsp;&nbsp; )<br><br>With these changes, all testcases run without error.<br><br>Cheers.&nbsp; Love mondrian.. just wish I understood more to be able to make more helpful contributions.. but I try - running through a debugger to find where does what is a pain.. but hey... I learn :)) .. java rulez :-)
<br><br>Ati<br><br><br><br><div><span class="gmail_quote">On 10/26/07, <b class="gmail_sendername">Julian Hyde</b> &lt;<a href="mailto:julianhyde@speakeasy.net">julianhyde@speakeasy.net</a>&gt; wrote:</span><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&#39;m not an expert on AggGen. If there are any experts out 
there, please chime in. Your changes sound reasonable, so I&#39;ve checked them in 
as change 10108.</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></div><br></blockquote></div><br>