----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/58604/#review172607 -----------------------------------------------------------
lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java Lines 1727 (patched) <https://reviews.apache.org/r/58604/#comment245692> Should have a proper error code associated and message should go to lens-errors.conf lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java Lines 2044-2046 (patched) <https://reviews.apache.org/r/58604/#comment245695> We can use FailingQueryDriver instead of changing server state like this. lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java Lines 2051-2054 (patched) <https://reviews.apache.org/r/58604/#comment245696> Can we call estimate api instead of this and validate the error response from api? lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java Lines 2065 (patched) <https://reviews.apache.org/r/58604/#comment245697> Calling sleeping estimates in for loop will do than running them as Runnables. - Amareshwari Sriramadasu On April 21, 2017, 7:25 a.m., Lavkesh Lahngir wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/58604/ > ----------------------------------------------------------- > > (Updated April 21, 2017, 7:25 a.m.) > > > Review request for lens. > > > Bugs: LENS-1413 > https://issues.apache.org/jira/browse/LENS-1413 > > > Repository: lens > > > Description > ------- > > 1. Cancel estimates for other drivers when the estimate pool queue is full. > 2. Return the appropriate error message > > > Diffs > ----- > > > lens-server-api/src/test/java/org/apache/lens/server/api/driver/MockDriver.java > 92724c6 > > lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java > c6fbeda > > lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java > 7d8c977 > lens-server/src/test/resources/lens-site.xml 334a9c4 > > > Diff: https://reviews.apache.org/r/58604/diff/1/ > > > Testing > ------- > > > Thanks, > > Lavkesh Lahngir > >