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




lens-regression/pom.xml (lines 81 - 86)
<https://reviews.apache.org/r/46206/#comment193415>

    Can we add to parent pom's dependency management and add here without 
version?



lens-regression/pom.xml (line 83)
<https://reviews.apache.org/r/46206/#comment193416>

    Why is protobuf required anyway?



lens-regression/src/main/java/org/apache/lens/regression/util/Util.java (lines 
229 - 232)
<https://reviews.apache.org/r/46206/#comment193417>

    Remove commented code.



lens-regression/src/test/java/org/apache/lens/regression/client/ITPreparedQueryTests.java
 (lines 271 - 287)
<https://reviews.apache.org/r/46206/#comment193418>

    Why are 10 second sleeps needed? This will slow down the tests.


- Rajat Khandelwal


On April 21, 2016, 1:06 p.m., Archana H wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/46206/
> -----------------------------------------------------------
> 
> (Updated April 21, 2016, 1:06 p.m.)
> 
> 
> Review request for lens and Rajat Khandelwal.
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Adding prepared and query API tests
> 
> 
> Diffs
> -----
> 
>   lens-regression/pom.xml 1a656e3 
>   
> lens-regression/src/main/java/org/apache/lens/regression/core/constants/DriverConfig.java
>  PRE-CREATION 
>   
> lens-regression/src/main/java/org/apache/lens/regression/core/constants/QueryInventory.java
>  eb889c4 
>   
> lens-regression/src/main/java/org/apache/lens/regression/core/helpers/QueryHelper.java
>  5035ce8 
>   
> lens-regression/src/main/java/org/apache/lens/regression/core/helpers/SessionHelper.java
>  64268a6 
>   
> lens-regression/src/main/java/org/apache/lens/regression/core/type/MapBuilder.java
>  eae8a06 
>   lens-regression/src/main/java/org/apache/lens/regression/util/Util.java 
> 6c3d234 
>   lens-regression/src/main/resources/lens.properties de00410 
>   lens-regression/src/main/resources/template.lens.properties PRE-CREATION 
>   
> lens-regression/src/test/java/org/apache/lens/regression/client/ITListQueryTest.java
>  821c388 
>   
> lens-regression/src/test/java/org/apache/lens/regression/client/ITPreparedQueryTests.java
>  PRE-CREATION 
>   
> lens-regression/src/test/java/org/apache/lens/regression/client/ITQueryApiTests.java
>  PRE-CREATION 
>   
> lens-regression/src/test/java/org/apache/lens/regression/client/SessionResourceTests.java
>  bbe438f 
> 
> Diff: https://reviews.apache.org/r/46206/diff/
> 
> 
> Testing
> -------
> 
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [2.624s]
> [INFO] Lens .............................................. SUCCESS [4.030s]
> [INFO] Lens API .......................................... SUCCESS [33.199s]
> [INFO] Lens API for server and extensions ................ SUCCESS [22.770s]
> [INFO] Lens Cube ......................................... SUCCESS 
> [11:51.946s]
> [INFO] Lens DB storage ................................... SUCCESS [21.493s]
> [INFO] Lens Query Library ................................ SUCCESS [16.262s]
> [INFO] Lens Hive Driver .................................. SUCCESS [3:17.615s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [39.777s]
> [INFO] Lens Elastic Search Driver ........................ SUCCESS [17.531s]
> [INFO] Lens Server ....................................... SUCCESS 
> [17:22.143s]
> [INFO] Lens client ....................................... SUCCESS [36.441s]
> [INFO] Lens CLI .......................................... SUCCESS [3:15.608s]
> [INFO] Lens Examples ..................................... SUCCESS [11.098s]
> [INFO] Lens Ship Jars to Distributed Cache ............... SUCCESS [0.774s]
> [INFO] Lens Distribution ................................. SUCCESS [9.812s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:23.875s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [1.760s]
> [INFO] Lens Regression ................................... SUCCESS [14.017s]
> [INFO] Lens UI ........................................... SUCCESS [46.735s]
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Total time: 41:50.603s
> [INFO] Finished at: Thu Apr 14 12:23:13 UTC 2016
> [INFO] Final Memory: 213M/2598M
> [INFO] 
> ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Archana H
> 
>

Reply via email to