[Mondrian] Aggregation.flush code review

Richard Emberson remberson at edgedynamics.com
Wed Apr 4 16:41:08 EDT 2007


Added "enter the"

Richard Emberson wrote:
> 
> It would seem that in the Aggregation
> flush(CacheControl,RolapCacheRegion) method, if
> boolean bitmapsIntersect is false, then
> there is no need to enter the segmentLoop for loop since
> 
>             if (!bitmapsIntersect) {
>                 continue;
>             }
> 
> always executes.
> 
> Yes?
> 
> 
> Richard
> 
> 
> -- 
> Quis custodiet ipsos custodes:
> This email message is for the sole use of the intended recipient(s) and
> may contain confidential information.  Any unauthorized review, use,
> disclosure or distribution is prohibited.  If you are not the intended
> recipient, please contact the sender by reply email and destroy all
> copies of the original message.
> _______________________________________________
> Mondrian mailing list
> Mondrian at pentaho.org
> http://lists.pentaho.org/mailman/listinfo/mondrian
> 


-- 
Quis custodiet ipsos custodes:
This email message is for the sole use of the intended recipient(s) and
may contain confidential information.  Any unauthorized review, use,
disclosure or distribution is prohibited.  If you are not the intended
recipient, please contact the sender by reply email and destroy all
copies of the original message.



More information about the Mondrian mailing list