> On May 20, 2016, 3:17 a.m., Puneet Gupta wrote: > > lens-regression/src/test/java/org/apache/lens/regression/throttling/Throttling.java, > > line 146 > > <https://reviews.apache.org/r/47593/diff/1/?file=1387941#file1387941line146> > > > > It may pick handleList.get(3) sometimes. Should we change the logic to > > check 2||3 is running
Since all queries are of same priority, will it not always pick handleList.get(2) first? > On May 20, 2016, 3:17 a.m., Puneet Gupta wrote: > > lens-regression/src/test/java/org/apache/lens/regression/throttling/Throttling.java, > > lines 163-165 > > <https://reviews.apache.org/r/47593/diff/1/?file=1387941#file1387941line163> > > > > How long will these queries run for ? If <30 secs, then the system will > > never have more than 3 queries across all states at any time. hive queries run for more than 30secs but jdbc completes with in 10secs. > On May 20, 2016, 3:17 a.m., Puneet Gupta wrote: > > lens-regression/src/test/java/org/apache/lens/regression/throttling/Throttling.java, > > line 180 > > <https://reviews.apache.org/r/47593/diff/1/?file=1387941#file1387941line180> > > > > Should maxConcurrent = max allowed on hive + max allowed on JDBC ? No. Running and queued list is filtering only hive queries > On May 20, 2016, 3:17 a.m., Puneet Gupta wrote: > > lens-regression/src/test/java/org/apache/lens/regression/throttling/Throttling.java, > > line 194 > > <https://reviews.apache.org/r/47593/diff/1/?file=1387941#file1387941line194> > > > > Should we wait or should we kill the queries to reduce test run time ? Better to wait. If there is any issue then queries will fail, which can be caught with status assert check. - Archana ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47593/#review134079 ----------------------------------------------------------- On June 2, 2016, 7:52 a.m., Archana H wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47593/ > ----------------------------------------------------------- > > (Updated June 2, 2016, 7:52 a.m.) > > > Review request for lens, Deepak Barr, Rajat Khandelwal, and Puneet Gupta. > > > Repository: lens > > > Description > ------- > > Throttling tests to regression > > > Diffs > ----- > > > lens-regression/src/test/java/org/apache/lens/regression/throttling/ITCostTests.java > 4da4c3a > > lens-regression/src/test/java/org/apache/lens/regression/throttling/Throttling.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/47593/diff/ > > > Testing > ------- > > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.563s] > [INFO] Lens .............................................. SUCCESS [3.553s] > [INFO] Lens API .......................................... SUCCESS [20.758s] > [INFO] Lens API for server and extensions ................ SUCCESS [16.842s] > [INFO] Lens Cube ......................................... SUCCESS > [14:01.494s] > [INFO] Lens DB storage ................................... SUCCESS [17.028s] > [INFO] Lens Query Library ................................ SUCCESS [18.060s] > [INFO] Lens Hive Driver .................................. SUCCESS [1:33.480s] > [INFO] Lens Driver for JDBC .............................. SUCCESS [38.171s] > [INFO] Lens Elastic Search Driver ........................ SUCCESS [23.389s] > [INFO] Lens Server ....................................... SUCCESS [8:49.961s] > [INFO] Lens client ....................................... SUCCESS [1:29.573s] > [INFO] Lens CLI .......................................... SUCCESS [1:18.608s] > [INFO] Lens Examples ..................................... SUCCESS [7.455s] > [INFO] Lens Ship Jars to Distributed Cache ............... SUCCESS [0.671s] > [INFO] Lens Distribution ................................. SUCCESS [9.060s] > [INFO] Lens ML Lib ....................................... SUCCESS [1:09.787s] > [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.321s] > [INFO] Lens Regression ................................... SUCCESS [11.454s] > [INFO] Lens UI ........................................... SUCCESS [27.047s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 31:41.125s > [INFO] Finished at: Thu May 19 11:53:17 UTC 2016 > [INFO] Final Memory: 186M/2279M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Archana H > >
