> On June 6, 2016, 5:30 a.m., Puneet Gupta wrote: > > lens-regression/src/test/java/org/apache/lens/regression/throttling/Throttling.java, > > line 449 > > <https://reviews.apache.org/r/47593/diff/2/?file=1404592#file1404592line449> > > > > Should we rename priority1,2 to low and verylow?
If we want to put filter on any other priorty, then name will conflict, hence kept the generic name. > On June 6, 2016, 5:30 a.m., Puneet Gupta wrote: > > lens-regression/src/test/java/org/apache/lens/regression/throttling/Throttling.java, > > lines 509-512 > > <https://reviews.apache.org/r/47593/diff/2/?file=1404592#file1404592line509> > > > > Can we also assert totalrunning <= maxConcurrent Its done just after getting runing and queued count. > On June 6, 2016, 5:30 a.m., Puneet Gupta wrote: > > lens-regression/src/test/java/org/apache/lens/regression/throttling/Throttling.java, > > lines 331-335 > > <https://reviews.apache.org/r/47593/diff/2/?file=1404592#file1404592line331> > > > > Should we chcek if queries 1 and 4 have started running after this ? Checking for the same. On June 6, 2016, 5:30 a.m., Archana H wrote: > > Are we adding usercost based throttling tests cases separately ? Yes. Few of them are added in ITCostsTests. Will add more usercost related to that. - Archana ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47593/#review136223 ----------------------------------------------------------- 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 > >
