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

Ship it!


Ship It!

- Amareshwari Sriramadasu


On Sept. 14, 2015, 1:05 p.m., Rajat Khandelwal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/38212/
> -----------------------------------------------------------
> 
> (Updated Sept. 14, 2015, 1:05 p.m.)
> 
> 
> Review request for lens.
> 
> 
> Bugs: LENS-392
>     https://issues.apache.org/jira/browse/LENS-392
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> e.g. for a monthly query with look ahead = 1, if next month partition is not 
> present, it looks for process time partitions for all days of the next month. 
> And all of them might not be present. So we should proceed with whichever 
> ones are present.
> 
> 
> Diffs
> -----
> 
>   lens-cube/src/main/java/org/apache/lens/cube/metadata/FactPartition.java 
> fc2d85b89e9809a61c982ca78a7d18ffacee76f6 
>   lens-cube/src/main/java/org/apache/lens/cube/metadata/TimePartition.java 
> d52f1686254ab0b9c977af84731888778edacb8a 
>   
> lens-cube/src/main/java/org/apache/lens/cube/metadata/TimePartitionRange.java 
> f5f8d4c8ec4177b2d83f5f567386a0422958c69d 
>   
> lens-cube/src/main/java/org/apache/lens/cube/metadata/timeline/RangesPartitionTimeline.java
>  1b9a44a6c8ac0280bc79d7e62753d692e8e4cd57 
>   
> lens-cube/src/main/java/org/apache/lens/cube/parse/AbridgedTimeRangeWriter.java
>  2caea569a6af1f717bef49491b024441808b5448 
>   
> lens-cube/src/main/java/org/apache/lens/cube/parse/StorageTableResolver.java 
> 58d0fa7b3d4b4ffd503b3606c3a42d4713d7ef11 
>   lens-cube/src/test/java/org/apache/lens/cube/parse/CubeTestSetup.java 
> 7f5629200f82f92e00fccd423097666383b06e18 
>   lens-cube/src/test/java/org/apache/lens/cube/parse/TestCubeRewriter.java 
> 7e5184c472b230cbee0c633191283a0368a82949 
> 
> Diff: https://reviews.apache.org/r/38212/diff/
> 
> 
> Testing
> -------
> 
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [2.285s]
> [INFO] Lens .............................................. SUCCESS [2.853s]
> [INFO] Lens API .......................................... SUCCESS [21.064s]
> [INFO] Lens API for server and extensions ................ SUCCESS [20.780s]
> [INFO] Lens Cube ......................................... SUCCESS [5:27.193s]
> [INFO] Lens DB storage ................................... SUCCESS [18.947s]
> [INFO] Lens Query Library ................................ SUCCESS [16.214s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:54.010s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [36.208s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [17.168s]
> [INFO] Lens Server ....................................... SUCCESS [5:37.934s]
> [INFO] Lens client ....................................... SUCCESS [38.626s]
> [INFO] Lens CLI .......................................... SUCCESS [2:39.371s]
> [INFO] Lens Examples ..................................... SUCCESS [8.829s]
> [INFO] Lens Distribution ................................. SUCCESS [9.163s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:21.146s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.040s]
> [INFO] Lens Regression ................................... SUCCESS [10.560s]
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Total time: 21:25.417s
> [INFO] Finished at: Mon Sep 14 12:59:16 UTC 2015
> [INFO] Final Memory: 194M/2078M
> [INFO] 
> ------------------------------------------------------------------------
> 
> Added a test case
> 
> 
> Thanks,
> 
> Rajat Khandelwal
> 
>

Reply via email to