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



lens-server/src/main/java/org/apache/lens/server/BaseLensService.java (line 281)
<https://reviews.apache.org/r/39576/#comment162941>

    What if it's already 0?


- Rajat Khandelwal


On Oct. 31, 2015, 5:54 a.m., Raju Bairishetti wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39576/
> -----------------------------------------------------------
> 
> (Updated Oct. 31, 2015, 5:54 a.m.)
> 
> 
> Review request for lens, Amareshwari Sriramadasu and Rajat Khandelwal.
> 
> 
> Bugs: LENS-833
>     https://issues.apache.org/jira/browse/LENS-833
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Failing the open session operation if user creates more sessions than 
> configured limit.
> 
> 
> Diffs
> -----
> 
>   lens-api/src/main/resources/lens-errors.conf a582dc2 
>   
> lens-server-api/src/main/java/org/apache/lens/server/api/LensConfConstants.java
>  7ee0749 
>   lens-server/src/main/java/org/apache/lens/server/BaseLensService.java 
> 0821fe7 
>   lens-server/src/main/resources/lensserver-default.xml 5f268cb 
>   lens-server/src/test/java/org/apache/lens/server/TestServerMode.java 
> def5683 
>   
> lens-server/src/test/java/org/apache/lens/server/auth/FooBarAuthenticationProvider.java
>  8e22837 
>   
> lens-server/src/test/java/org/apache/lens/server/metastore/TestMetastoreService.java
>  ca0fe56 
>   
> lens-server/src/test/java/org/apache/lens/server/query/QueryAPIErrorResponseTest.java
>  18a8c8d 
>   lens-server/src/test/java/org/apache/lens/server/query/TestLensDAO.java 
> bc1463f 
>   
> lens-server/src/test/java/org/apache/lens/server/session/TestSessionResource.java
>  3055ce5 
>   
> lens-server/src/test/java/org/apache/lens/server/ui/TestSessionUIResource.java
>  6f7c216 
>   src/site/apt/admin/config.apt 88c1489 
> 
> Diff: https://reviews.apache.org/r/39576/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Raju Bairishetti
> 
>

Reply via email to