> On May 13, 2016, 5 a.m., Puneet Gupta wrote: > > lens-regression/src/test/java/org/apache/lens/regression/throttling/ITCostTests.java, > > line 261 > > <https://reviews.apache.org/r/47301/diff/1/?file=1381195#file1381195line261> > > > > Should we combine this test case with > > testCostCeilingWithProrityMaxConcurrent ?
In testCostCeilingWithProrityMaxConcurrent boundary values gets validated, whether count is exactly <= thresold, where as in diffUserConcurrentPriorityThrottling count may be way less than thresold. - Archana ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47301/#review133031 ----------------------------------------------------------- On May 16, 2016, 11:24 a.m., Archana H wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47301/ > ----------------------------------------------------------- > > (Updated May 16, 2016, 11:24 a.m.) > > > Review request for lens, Amareshwari Sriramadasu, Deepak Barr, Rajat > Khandelwal, and Puneet Gupta. > > > Repository: lens > > > Description > ------- > > Adding user cost ceiling tests > > > Diffs > ----- > > lens-regression/pom.xml b9841e8 > > lens-regression/src/main/java/org/apache/lens/regression/core/constants/DriverConfig.java > c0fc15a > lens-regression/src/main/resources/queries.properties PRE-CREATION > > lens-regression/src/test/java/org/apache/lens/regression/ITStreamingTests.java > d942e3c > > lens-regression/src/test/java/org/apache/lens/regression/config/ITServerConfigTests.java > f326a4f > > lens-regression/src/test/java/org/apache/lens/regression/config/ITSessionConfigTests.java > d3b8ffb > > lens-regression/src/test/java/org/apache/lens/regression/throttling/ITCostTests.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/47301/diff/ > > > Testing > ------- > > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.978s] > [INFO] Lens .............................................. SUCCESS [3.996s] > [INFO] Lens API .......................................... SUCCESS [20.521s] > [INFO] Lens API for server and extensions ................ SUCCESS [16.392s] > [INFO] Lens Cube ......................................... SUCCESS > [14:46.227s] > [INFO] Lens DB storage ................................... SUCCESS [16.187s] > [INFO] Lens Query Library ................................ SUCCESS [13.004s] > [INFO] Lens Hive Driver .................................. SUCCESS [1:22.171s] > [INFO] Lens Driver for JDBC .............................. SUCCESS [39.077s] > [INFO] Lens Elastic Search Driver ........................ SUCCESS [23.482s] > [INFO] Lens Server ....................................... SUCCESS [8:42.013s] > [INFO] Lens client ....................................... SUCCESS [1:30.298s] > [INFO] Lens CLI .......................................... SUCCESS [1:17.992s] > [INFO] Lens Examples ..................................... SUCCESS [8.909s] > [INFO] Lens Ship Jars to Distributed Cache ............... SUCCESS [0.644s] > [INFO] Lens Distribution ................................. SUCCESS [9.850s] > [INFO] Lens ML Lib ....................................... SUCCESS [1:08.855s] > [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.197s] > [INFO] Lens Regression ................................... SUCCESS [10.979s] > [INFO] Lens UI ........................................... SUCCESS [42.076s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 32:17.865s > [INFO] Finished at: Thu May 12 12:03:42 UTC 2016 > [INFO] Final Memory: 195M/2815M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Archana H > >
