----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/45273/#review125242 -----------------------------------------------------------
lens-server-api/src/main/java/org/apache/lens/server/api/events/AsyncEventListener.java (line 94) <https://reviews.apache.org/r/45273/#comment188036> Do we need to increase the default keep alive time to avoid frequent thread creations and terminations? - Raju Bairishetti On March 24, 2016, 6:36 a.m., Amareshwari Sriramadasu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/45273/ > ----------------------------------------------------------- > > (Updated March 24, 2016, 6:36 a.m.) > > > Review request for lens. > > > Bugs: LENS-993 > https://issues.apache.org/jira/browse/LENS-993 > > > Repository: lens > > > Description > ------- > > - Using SynchronousQueue as in Executors.newCachedThreadPool, but pass a max > pool size for estimate pool > - Using Fixed Pool for AsyncEventListeners > > > Diffs > ----- > > > lens-server-api/src/main/java/org/apache/lens/server/api/events/AsyncEventListener.java > 84728e5 > > lens-server/src/main/java/org/apache/lens/server/query/QueryEndNotifier.java > 63c38d9 > > lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java > 90c25e4 > lens-server/src/main/java/org/apache/lens/server/query/ResultFormatter.java > 9955278 > > lens-server/src/test/java/org/apache/lens/server/query/TestEventService.java > a2ca17f > > Diff: https://reviews.apache.org/r/45273/diff/ > > > Testing > ------- > > > Thanks, > > Amareshwari Sriramadasu > >
