> On May 25, 2016, 5:54 a.m., Puneet Gupta wrote: > > lens-server-api/src/main/java/org/apache/lens/server/api/driver/WeightCostSelector.java, > > lines 46-48 > > <https://reviews.apache.org/r/47777/diff/1/?file=1392428#file1392428line46> > > > > Better to check explicitly is debug logs are enabled before going into > > this loop . In most cases this loop can be skipped. > > Anshul Gupta wrote: > Added check for whether debug logs are enabled. > Added driver ratio in the debug logs.
Did not find the check in latest path . Please check > On May 25, 2016, 5:54 a.m., Puneet Gupta wrote: > > lens-server-api/src/main/java/org/apache/lens/server/api/driver/WeightCostSelector.java, > > line 51 > > <https://reviews.apache.org/r/47777/diff/1/?file=1392428#file1392428line51> > > > > Should we optimize in case only one driver is selected ? We can skip > > all the extra processing in this case > > Anshul Gupta wrote: > Optimized for the same. Did not find it in latest patch. > On May 25, 2016, 5:54 a.m., Puneet Gupta wrote: > > lens-server-api/src/main/java/org/apache/lens/server/api/driver/WeightCostSelector.java, > > line 61 > > <https://reviews.apache.org/r/47777/diff/1/?file=1392428#file1392428line61> > > > > An arry list should suffice here. Do we need a set ? > > ctx.getDriverContext().getDriversWithValidQueryCost() should return unique > > drivers. > > Anshul Gupta wrote: > getDriverWithValidQueyCost already uses a set to return unique drivers. > Changed to arraylist. Did not find this in latest patch. Please check > On May 25, 2016, 5:54 a.m., Puneet Gupta wrote: > > lens-server-api/src/main/java/org/apache/lens/server/api/driver/WeightCostSelector.java, > > line 63 > > <https://reviews.apache.org/r/47777/diff/1/?file=1392428#file1392428line63> > > > > Should we cache the cost for the minCostDriver and use it for comparing > > ? This will save extra lookup Did not find the fix in patch. Please check. > On May 25, 2016, 5:54 a.m., Puneet Gupta wrote: > > lens-server-api/src/main/java/org/apache/lens/server/api/query/AbstractQueryContext.java, > > line 492 > > <https://reviews.apache.org/r/47777/diff/1/?file=1392429#file1392429line492> > > > > AbstractQueryContext.java can be removed form chnage list. See no > > chnages here Not removed in latest patch. Please check. - Puneet ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47777/#review134702 ----------------------------------------------------------- On May 25, 2016, 11:22 a.m., Anshul Gupta wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47777/ > ----------------------------------------------------------- > > (Updated May 25, 2016, 11:22 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/TestWeightedCostSelector.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/driver/WeightCostSelector.java > PRE-CREATION > > lens-server-api/src/main/java/org/apache/lens/server/api/query/AbstractQueryContext.java > e160f58bfb45cffb05b9f6be19db3bc7605dde57 > > 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 > >
