----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47593/#review134079 -----------------------------------------------------------
Partial review. lens-regression/src/test/java/org/apache/lens/regression/throttling/Throttling.java (line 146) <https://reviews.apache.org/r/47593/#comment198723> It may pick handleList.get(3) sometimes. Should we change the logic to check 2||3 is running lens-regression/src/test/java/org/apache/lens/regression/throttling/Throttling.java (lines 163 - 165) <https://reviews.apache.org/r/47593/#comment198728> 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. lens-regression/src/test/java/org/apache/lens/regression/throttling/Throttling.java (line 180) <https://reviews.apache.org/r/47593/#comment198726> Should maxConcurrent = max allowed on hive + max allowed on JDBC ? lens-regression/src/test/java/org/apache/lens/regression/throttling/Throttling.java (line 194) <https://reviews.apache.org/r/47593/#comment198727> Should we wait or should we kill the queries to reduce test run time ? - Puneet Gupta On May 19, 2016, 2:26 p.m., Archana H wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47593/ > ----------------------------------------------------------- > > (Updated May 19, 2016, 2:26 p.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 > >
