> On May 25, 2016, 12:50 p.m., Rajat Khandelwal wrote: > > lens-cube/src/test/java/org/apache/lens/driver/cube/TestWeightedCostSelector.java, > > lines 28-34 > > <https://reviews.apache.org/r/47777/diff/2/?file=1393439#file1393439line28> > > > > It's not using any of `lens-cube` dependencies, so this test class can > > move to `lens-server-api` module. Same should be possible with > > `TestMinCostSelector`. Let's move both there.
Classes moved to lens-server-api module. > On May 25, 2016, 12:50 p.m., Rajat Khandelwal wrote: > > lens-cube/src/test/java/org/apache/lens/driver/cube/TestWeightedCostSelector.java, > > line 66 > > <https://reviews.apache.org/r/47777/diff/2/?file=1393439#file1393439line66> > > > > This method is performing 2 tests sequentially. The problem with that > > is that if the first one fails, the second one never gets to run. This > > method should be split into multiple methods. Ideally, one method per test > > should be followed. > > > > > > 1. 3 Drivers with ratio -- Should distribute in ratio. > > 2. 3 Drivers without ratio -- Should distribute equally. > > 3. 2 Drivers, where cost of queries on one driver is always less than > > the other one -- The first one should always be picked. > > > > For 1 and 2, let's have one extra driver which always gives a high > > cost. Assertions will be that that driver should never get picked. Added three test cases along with another test case for the default MinCostDriver behavior. - Anshul ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47777/#review134760 ----------------------------------------------------------- On May 26, 2016, 10:07 a.m., Anshul Gupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47777/ > ----------------------------------------------------------- > > (Updated May 26, 2016, 10:07 a.m.) > > > Review request for lens. > > > Bugs: LENS-661 > https://issues.apache.org/jira/browse/LENS-661 > > > Repository: lens > > > Description > ------- > > Added Weighted selector > > > Diffs > ----- > > > lens-cube/src/test/java/org/apache/lens/driver/cube/TestMinCostSelector.java > 82672294fcbf462db1fd75f00e796d5dae9d2c22 > > lens-server-api/src/main/java/org/apache/lens/server/api/LensConfConstants.java > 85f54d1f835f86dd6122a987525853e0ffa9488c > > lens-server-api/src/main/java/org/apache/lens/server/api/driver/WeightedQueryCostDriverSelector.java > PRE-CREATION > > lens-server-api/src/test/java/org/apache/lens/server/api/driver/TestMinCostSelector.java > PRE-CREATION > > lens-server-api/src/test/java/org/apache/lens/server/api/driver/TestWeightedCostSelector.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/47777/diff/ > > > Testing > ------- > > Here's the reactor summary after running mvn clean package > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Lens Server ........................................ SUCCESS [11:32 > min] > [INFO] Lens client ........................................ SUCCESS [01:49 > min] > [INFO] Lens CLI ........................................... SUCCESS [02:02 > min] > [INFO] Lens Examples ...................................... SUCCESS [ 8.430 > s] > [INFO] Lens Ship Jars to Distributed Cache ................ SUCCESS [ 2.801 > s] > [INFO] Lens Distribution .................................. SUCCESS [ 21.429 > s] > [INFO] Lens ML Lib ........................................ SUCCESS [02:28 > min] > [INFO] Lens ML Ext Distribution ........................... SUCCESS [ 20.690 > s] > [INFO] Lens Regression .................................... SUCCESS [ 9.313 > s] > [INFO] Lens UI ............................................ SUCCESS [ 14.163 > s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 19:10 min > [INFO] Finished at: 2016-05-24T20:31:02+05:30 > [INFO] Final Memory: 103M/466M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Anshul Gupta > >
