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

(Updated July 21, 2016, 11:56 a.m.)


Review request for lens.


Changes
-------

Review comments incorporated


Bugs: LENS-1168
    https://issues.apache.org/jira/browse/LENS-1168


Repository: lens


Description
-------

Changes include :
- Add query launcher pool which would launch queries after applying constraint 
checks.
- Add max concurrent launching queries constraint - to limit number parallel 
launches at any time.
- Update StatusPoller to skip launching queries
- Update cancelQuery to interrupt launcher.
- Added test to verify parallel launches.


Diffs (updated)
-----

  
lens-server-api/src/main/java/org/apache/lens/server/api/LensConfConstants.java 
bd9b1ab 
  
lens-server-api/src/main/java/org/apache/lens/server/api/metrics/MetricsService.java
 7fd2d81 
  
lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryContext.java
 8ba0689 
  
lens-server-api/src/main/java/org/apache/lens/server/api/query/QueryExecutionService.java
 d10ad09 
  
lens-server-api/src/main/java/org/apache/lens/server/api/query/constraint/MaxConcurrentDriverQueriesConstraint.java
 b2319a9 
  
lens-server-api/src/main/java/org/apache/lens/server/api/query/constraint/MaxConcurrentDriverQueriesConstraintFactory.java
 442cd99 
  
lens-server-api/src/test/java/org/apache/lens/server/api/query/constraint/MaxConcurrentDriverQueriesConstraintTest.java
 38b74ae 
  
lens-server/src/main/java/org/apache/lens/server/metrics/MetricsServiceImpl.java
 1e8d540 
  
lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java
 be0e2c8 
  lens-server/src/main/java/org/apache/lens/server/session/LensSessionImpl.java 
e77c7fa 
  lens-server/src/main/resources/lensserver-default.xml 3ce8fc1 
  lens-server/src/test/java/org/apache/lens/server/LensJerseyTest.java d5e975a 
  
lens-server/src/test/java/org/apache/lens/server/common/FailingQueryDriver.java 
0b38517 
  
lens-server/src/test/java/org/apache/lens/server/query/TestQueryConstraints.java
 0df436a 
  lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java 
6af4225 
  
lens-server/src/test/java/org/apache/lens/server/query/constraint/ThreadSafeEstimatedQueryCollectionTest.java
 c8ebd0c 
  src/site/apt/admin/config.apt 852955b 

Diff: https://reviews.apache.org/r/49371/diff/


Testing
-------


Thanks,

Amareshwari Sriramadasu

Reply via email to