> 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 > > Archana H wrote: > Since all queries are of same priority, will it not always pick > handleList.get(2) first? > > Puneet Gupta wrote: > In rare cases this may not hold good since we check for canLauch() in a > loop and a query can finish any time. If there is an easy way to counter this > in test case, it will be foolproof.
Will add it, if this scenario is observed anytime. - Archana ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47593/#review134079 ----------------------------------------------------------- On June 2, 2016, 8:40 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, 8:40 a.m.) > > > Review request for lens, Deepak Barr, Rajat Khandelwal, and Puneet Gupta. > > > Repository: lens > > > Description > ------- > > Throttling tests to regression > > > Diffs > ----- > > lens-regression/pom.xml b9bee97 > lens-regression/src/main/java/org/apache/lens/regression/util/Util.java > 1a538da > > lens-regression/src/test/java/org/apache/lens/regression/client/KillQueryTests.java > e588c1b > > 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 > >
