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




lens-server/src/main/java/org/apache/lens/server/LensServer.java (line 102)
<https://reviews.apache.org/r/51175/#comment212330>

    ThreadpoolConfig which comes with default implementation is handling 
*uncaughtexceptions*. Can we add **threadFactory to threadPoolConfig** here as 
well? 
    Refer the following link 
    
https://github.com/jersey/jersey/blob/master/containers/grizzly2-http/src/main/java/org/glassfish/jersey/grizzly2/httpserver/GrizzlyHttpServerFactory.java#L268



lens-server/src/main/resources/lensserver-default.xml (line 898)
<https://reviews.apache.org/r/51175/#comment212331>

    Regenerate docs with these properties?


- Raju Bairishetti


On Aug. 17, 2016, 10:44 a.m., Deepak Barr wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51175/
> -----------------------------------------------------------
> 
> (Updated Aug. 17, 2016, 10:44 a.m.)
> 
> 
> Review request for lens.
> 
> 
> Bugs: LENS-1277
>     https://issues.apache.org/jira/browse/LENS-1277
> 
> 
> Repository: lens
> 
> 
> Description
> -------
> 
> Unlike other servers, Reconfiguring grizzly threadpool requires a java code 
> change.
> 
> 
> Diffs
> -----
> 
>   
> lens-server-api/src/main/java/org/apache/lens/server/api/LensConfConstants.java
>  57d13f9 
>   lens-server/src/main/java/org/apache/lens/server/LensServer.java 203c249 
>   lens-server/src/main/resources/lensserver-default.xml 460e3dd 
> 
> Diff: https://reviews.apache.org/r/51175/diff/
> 
> 
> Testing
> -------
> 
> Yes.
> 
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO]
> [INFO] Lens Checkstyle Rules .............................. SUCCESS [  1.606 
> s]
> [INFO] Lens ............................................... SUCCESS [  2.867 
> s]
> [INFO] Lens API ........................................... SUCCESS [ 17.072 
> s]
> [INFO] Lens API for server and extensions ................. SUCCESS [ 15.527 
> s]
> [INFO] Lens Cube .......................................... SUCCESS [13:04 
> min]
> [INFO] Lens DB storage .................................... SUCCESS [ 11.913 
> s]
> [INFO] Lens Query Library ................................. SUCCESS [  9.820 
> s]
> [INFO] Lens Hive Driver ................................... SUCCESS [21:45 
> min]
> [INFO] Lens Driver for JDBC ............................... SUCCESS [ 25.224 
> s]
> [INFO] Lens Elastic Search Driver ......................... SUCCESS [ 13.784 
> s]
> [INFO] Lens Server ........................................ SUCCESS [13:16 
> min]
> [INFO] Lens client ........................................ SUCCESS [01:19 
> min]
> [INFO] Lens CLI ........................................... SUCCESS [01:02 
> min]
> [INFO] Lens Examples ...................................... SUCCESS [  5.496 
> s]
> [INFO] Lens Ship Jars to Distributed Cache ................ SUCCESS [  0.523 
> s]
> [INFO] Lens Distribution .................................. SUCCESS [  9.798 
> s]
> [INFO] Lens ML Lib ........................................ SUCCESS [ 56.807 
> s]
> [INFO] Lens ML Ext Distribution ........................... SUCCESS [  2.532 
> s]
> [INFO] Lens Regression .................................... SUCCESS [  8.783 
> s]
> [INFO] Lens UI ............................................ SUCCESS [  3.418 
> s]
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Total time: 53:35 min
> [INFO] Finished at: 2016-08-17T14:02:19+05:30
> [INFO] Final Memory: 181M/1690M
> [INFO] 
> ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Deepak Barr
> 
>

Reply via email to