PDA

View Full Version : [Mondrian] RE: Eigenbase perforce change 8900 for review



Julian Hyde
03-21-2007, 05:11 AM
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 (DOT) 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 (DOT) 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
> >
>

_______________________________________________
Mondrian mailing list
Mondrian (AT) pentaho (DOT) org
http://lists.pentaho.org/mailman/listinfo/mondrian