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

Ship it!



lens-server-api/src/main/java/org/apache/lens/server/api/user/UserConfigLoader.java
<https://reviews.apache.org/r/34876/#comment140096>

    Should we change the log to debug ?


- Amareshwari Sriramadasu


On June 11, 2015, 12:18 p.m., Rajat Khandelwal wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/34876/
> -----------------------------------------------------------
> 
> (Updated June 11, 2015, 12:18 p.m.)
> 
> 
> Review request for lens.
> 
> 
> Bugs: LENS-555
>     https://issues.apache.org/jira/browse/LENS-555
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Right now the hadoop queue where the hive job is triggered is fixed. This 
> needs to be dynamic. We have one requirement where our yarn cluster has 
> hierarchical queues based on job priority. We need capability to be able to 
> choose the right queue based on priority of the job
> 
> 
> Diffs
> -----
> 
>   lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveDriver.java 
> 052d0e2bd0584e65dd06c0241dcf99dc1773045b 
>   
> lens-driver-hive/src/test/java/org/apache/lens/driver/hive/TestHiveDriver.java
>  b21258a1386327e51c6a34e6e7abb69c94b9c965 
>   
> lens-driver-hive/src/test/java/org/apache/lens/driver/hive/TestRemoteHiveDriver.java
>  b4086b858f2df8bfb6ea60ca216da8bb12a9cf5f 
>   lens-driver-jdbc/pom.xml a7994c5c6444ec6aa4b92b953784b16c2f8a3489 
>   lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/JDBCDriver.java 
> 70deaa401fa5168ad7780804ef68cca8a2660eae 
>   
> lens-driver-jdbc/src/test/java/org/apache/lens/driver/jdbc/TestJdbcDriver.java
>  2e3dcc7c5805733dd3784619c10e17e4690a4dca 
>   
> lens-server-api/src/main/java/org/apache/lens/server/api/driver/LensDriver.java
>  8511ce63b3845973d7532ffa15c7158e69726716 
>   
> lens-server-api/src/main/java/org/apache/lens/server/api/user/UserConfigLoader.java
>  PRE-CREATION 
>   
> lens-server-api/src/main/java/org/apache/lens/server/api/user/UserConfigLoaderException.java
>  PRE-CREATION 
>   
> lens-server-api/src/test/java/org/apache/lens/server/api/driver/MockDriver.java
>  30c1a71a46ecba88c05bc8ed5020c7f79aedf68d 
>   
> lens-server-api/src/test/java/org/apache/lens/server/api/user/MockUserConfigLoader.java
>  PRE-CREATION 
>   lens-server/src/main/java/org/apache/lens/server/LensServices.java 
> c27717fc2846a947af825c350c09bb2ac13e005d 
>   
> lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java
>  7623743a816aea6b9d5c95e69943fa2f95ca6c74 
>   
> lens-server/src/main/java/org/apache/lens/server/user/CustomUserConfigLoader.java
>  9a6a350a2808bb844ef870089fccf20a2136de72 
>   
> lens-server/src/main/java/org/apache/lens/server/user/DatabaseUserConfigLoader.java
>  d2679488aeb2b91105f087e34143bc848f1f3d52 
>   
> lens-server/src/main/java/org/apache/lens/server/user/FixedUserConfigLoader.java
>  b81b994ad00d09f5a0b9dd44cbdf731da3b2d0e6 
>   
> lens-server/src/main/java/org/apache/lens/server/user/LDAPBackedDatabaseUserConfigLoader.java
>  108a0123a78736892a3adb8bb24ff401e5a81b4e 
>   
> lens-server/src/main/java/org/apache/lens/server/user/PropertyBasedUserConfigLoader.java
>  16a9a709d097c8af8a84ea81282d0d43947fcd3d 
>   lens-server/src/main/java/org/apache/lens/server/user/UserConfigLoader.java 
> 21eea3bdbd5a84770e6144d1ed45806ad4dc6ee3 
>   
> lens-server/src/main/java/org/apache/lens/server/user/UserConfigLoaderException.java
>  0899746f8350b4b84c1e03c3f262f553faef9817 
>   
> lens-server/src/main/java/org/apache/lens/server/user/UserConfigLoaderFactory.java
>  383aaf5f0d5a3937b95b0077491d30678ffe46a3 
>   
> lens-server/src/test/java/org/apache/lens/server/user/FooBarConfigLoader.java 
> 5b3344003efa201e6f56357ba8813603185b7ec5 
>   
> lens-server/src/test/java/org/apache/lens/server/user/TestUserConfigLoader.java
>  4f399c27836303b487aef278a3b04bcbc97c7ff9 
> 
> Diff: https://reviews.apache.org/r/34876/diff/
> 
> 
> Testing
> -------
> 
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [3.279s]
> [INFO] Lens .............................................. SUCCESS [3.888s]
> [INFO] Lens API .......................................... SUCCESS [21.529s]
> [INFO] Lens API for server and extensions ................ SUCCESS [23.911s]
> [INFO] Lens Cube ......................................... SUCCESS [3:39.754s]
> [INFO] Lens DB storage ................................... SUCCESS [21.301s]
> [INFO] Lens Query Library ................................ SUCCESS [16.096s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:57.508s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [34.564s]
> [INFO] Lens Server ....................................... SUCCESS [5:21.248s]
> [INFO] Lens client ....................................... SUCCESS [35.348s]
> [INFO] Lens CLI .......................................... SUCCESS [4:51.950s]
> [INFO] Lens Examples ..................................... SUCCESS [13.121s]
> [INFO] Lens Distribution ................................. SUCCESS [10.219s]
> [INFO] Lens ML Lib ....................................... SUCCESS [1:33.169s]
> [INFO] Lens ML Ext Distribution .......................... SUCCESS [1.729s]
> [INFO] Lens Regression ................................... SUCCESS [11.151s]
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Total time: 21:40.865s
> [INFO] Finished at: Thu Jun 11 12:12:59 UTC 2015
> [INFO] Final Memory: 168M/1187M
> [INFO] 
> ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Rajat Khandelwal
> 
>

Reply via email to