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

(Updated May 3, 2016, 2:48 a.m.)


Review request for lens.


Changes
-------

- Updated ThreadSafeEstimatedQueryCollectionTest.java to use the new 
MaxConcurrentDriverQueriesConstraint constructor.
- Ran and complete build with test cases : pass


Bugs: lens-1027
    https://issues.apache.org/jira/browse/lens-1027


Repository: lens


Description
-------

As of now we can throttle queries via  Constraint  
driver.max.concurrent.launched.queries.per.queue which is specified as 
{"QueueName=MaxQueriesAllowed"},+.

Example driver.max.concurrent.launched.queries.per.queue = "QueueA=4,QueueB=3". 

We should be able to specify ANY/* as a queue in which case the value for ANY 
should be the default value for all queues that are not mentioned as part of 
the constraint.


Diffs (updated)
-----

  
lens-server-api/src/main/java/org/apache/lens/server/api/query/constraint/MaxConcurrentDriverQueriesConstraint.java
 58ebd9a 
  
lens-server-api/src/main/java/org/apache/lens/server/api/query/constraint/MaxConcurrentDriverQueriesConstraintFactory.java
 6db7da7 
  
lens-server-api/src/test/java/org/apache/lens/server/api/query/constraint/MaxConcurrentDriverQueriesConstraintTest.java
 4031122 
  
lens-server/src/test/java/org/apache/lens/server/query/constraint/ThreadSafeEstimatedQueryCollectionTest.java
 e1bf350 

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


Testing (updated)
-------

[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO] 
[INFO] Lens Checkstyle Rules ............................. SUCCESS [1.739s]
[INFO] Lens .............................................. SUCCESS [4.659s]
[INFO] Lens API .......................................... SUCCESS [21.046s]
[INFO] Lens API for server and extensions ................ SUCCESS [15.359s]
[INFO] Lens Cube ......................................... SUCCESS [7:08.125s]
[INFO] Lens DB storage ................................... SUCCESS [15.988s]
[INFO] Lens Query Library ................................ SUCCESS [11.192s]
[INFO] Lens Hive Driver .................................. SUCCESS [2:26.994s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [31.075s]
[INFO] Lens Elastic Search Driver ........................ SUCCESS [12.790s]
[INFO] Lens Server ....................................... SUCCESS [14:06.701s]
[INFO] Lens client ....................................... SUCCESS [1:24.310s]
[INFO] Lens CLI .......................................... SUCCESS [2:11.890s]
[INFO] Lens Examples ..................................... SUCCESS [8.929s]
[INFO] Lens Ship Jars to Distributed Cache ............... SUCCESS [0.752s]
[INFO] Lens Distribution ................................. SUCCESS [8.907s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:07.475s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [1.494s]
[INFO] Lens Regression ................................... SUCCESS [10.295s]
[INFO] Lens UI ........................................... SUCCESS [44.853s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 31:35.473s


Thanks,

Puneet Gupta

Reply via email to