-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/28555/#review63494
-----------------------------------------------------------



lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryContext.java
<https://reviews.apache.org/r/28555/#comment105760>

    I didnt understand the purpose of this. Is this for multi fact query?



lens-cube/src/main/java/org/apache/lens/cube/parse/StorageTableResolver.java
<https://reviews.apache.org/r/28555/#comment105761>

    You shoudnt remove timedimensions directly. Should make a copy.
    
    PartCol and timeddimension can be different


- Amareshwari Sriramadasu


On Dec. 2, 2014, 6:23 a.m., Rajat Khandelwal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/28555/
> -----------------------------------------------------------
> 
> (Updated Dec. 2, 2014, 6:23 a.m.)
> 
> 
> Review request for lens.
> 
> 
> Bugs: LENS-33
>     https://issues.apache.org/jira/browse/LENS-33
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> checkpoint
> 
> 
> checkpoint
> 
> 
> Merge branch 'master' of github.com:apache/incubator-lens into LENS-33
> 
> 
> checkpoint
> 
> 
> merge master
> 
> 
> Merge branch 'master' of github.com:apache/incubator-lens into LENS-33
> 
> 
> Merge branch 'master' of github.com:apache/incubator-lens into LENS-33
> 
> 
> checkpoint WIP
> 
> 
> Diffs
> -----
> 
>   lens-cube/src/main/java/org/apache/lens/cube/metadata/Storage.java 0a6f489 
>   lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryContext.java 
> 2409db8 
>   lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryRewriter.java 
> 1325c05 
>   lens-cube/src/main/java/org/apache/lens/cube/parse/DimHQLContext.java 
> 0070ec2 
>   lens-cube/src/main/java/org/apache/lens/cube/parse/DimOnlyHQLContext.java 
> 88a1b7d 
>   lens-cube/src/main/java/org/apache/lens/cube/parse/FactHQLContext.java 
> a7819c1 
>   
> lens-cube/src/main/java/org/apache/lens/cube/parse/SingleFactHQLContext.java 
> 8b4756f 
>   
> lens-cube/src/main/java/org/apache/lens/cube/parse/StorageTableResolver.java 
> 3fb9d09 
>   lens-cube/src/main/java/org/apache/lens/cube/parse/StorageUtil.java 01cfcf7 
> 
> Diff: https://reviews.apache.org/r/28555/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Rajat Khandelwal
> 
>

Reply via email to