[Mondrian] RE: Eigenbase perforce change 8900 for review

[Julian Hyde]

Richard,

You overwrote the change I made to MondrianProperties.java this morning.
Can you fix it please?

Julian


 

> -----Original Message-----
> From: Richard Emberson [mailto:remberson at edgedynamics.com] 
> Sent: Tuesday, March 20, 2007 4:30 PM
> To: Andreas Voss; Bart Pappyn; Julian Hyde; John V. Sichi; 
> Matt Campbell; Sam Birney
> Subject: Eigenbase perforce change 8900 for review
> 
> http://p4web.eigenbase.org/@md=d&c=6PU@//8900?ac=10
> 
> Change 8900 by emberson at bortei.head on 2007/03/20 15:28:08
> 
> 	MONDRIAN
> 	   Revised RolapResult evaluation algorithm and the 
> CrossJoin non-empty
> 	   optimization algorithm.
> 	   The CrossJoin change was to make the optimizer less 
> optimistic so that
> 	   it would include values that occur in corner cases. It also
> 	   cached information between invocations.
> 	   Added ability for Calc's to cache evaluation data in 
> the Query object.
> 	   Made the ResultLimit check also work when an 
> Iterable, rather than a
> 	   List, is used to generate the RolapAxis object.
> 	   Enable test RolapResultTest.java testD2
> 
> Affected files ...
> 
> ==== 
> //open/mondrian/src/main/mondrian/olap/MondrianProperties.java
> #70 (ktext) ====
> 
> 2c2
> < // $Id: 
> //open/mondrian/src/main/mondrian/olap/MondrianProperties.java#69 $
> ---
> > // $Id: 
> //open/mondrian/src/main/mondrian/olap/MondrianProperties.java#70 $
> 58c58
> <  * @version $Id: 
> //open/mondrian/src/main/mondrian/olap/MondrianProperties.java#69 $
> ---
> >  * @version $Id: 
> //open/mondrian/src/main/mondrian/olap/MondrianProperties.java#70 $
> 498,501c498,501
> <      *   <code>(possible -
> <      *   {@link #SparseSegmentCountThreshold countThreshold}) *
> <      *   {@link #SparseSegmentDensityThreshold 
> densityThreshold} &gt;
> <      *   actual</code>
> ---
> >      *   <code>possible -
> >      *   {@link #SparseSegmentCountThreshold countThreshold} *
> >      *   actual &gt;
> >      *   {@link #SparseSegmentDensityThreshold 
> densityThreshold}</code>
> 923,937d922
> < 
> <     /**
> <      * The crossjoin optimizer is flawed. It evaluates 
> potential cells
> <      * using default members when a member is not explicitly set in
> <      * the particular axis. This works unless another axis, 
> normal or
> <      * slicer, uses a member other than the default.
> <      * Setting this property to <code>true</code> will enable the
> <      * RolapResult to collect these implied members and if any are
> <      * found, then re-evaluate using the discovered members.
> <      * <p>
> <      * If you do not enable this property, then the 
> crossjoin optimizer,
> <      * if used, can produce errors. Bad results are 
> produced for some
> <      * queries and some data sets - its a combination of both so the
> <      * error does not manifest itself all the time.
> <      */
> 939a925
> > 
>