----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35063/#review87136 -----------------------------------------------------------
Ship it! Ship It! - Jaideep dhok On June 5, 2015, 9:03 a.m., Amareshwari Sriramadasu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/35063/ > ----------------------------------------------------------- > > (Updated June 5, 2015, 9:03 a.m.) > > > Review request for lens and Rajat Khandelwal. > > > Bugs: LENS-414 > https://issues.apache.org/jira/browse/LENS-414 > > > Repository: lens > > > Description > ------- > > Moved LightestFactResolver before LeastPartitionResolver - that says "if two > facts have the same least weight, then the fact with least number of time > partitions queried will be picked". This will be useful, if users did not set > fact weights. > > > Diffs > ----- > > lens-cube/src/main/java/org/apache/lens/cube/parse/CubeQueryRewriter.java > 57c91e7 > > lens-cube/src/main/java/org/apache/lens/cube/parse/LeastPartitionResolver.java > d0b3ccb > lens-cube/src/test/java/org/apache/lens/cube/parse/TestQueryMetrics.java > 39f70eb > > Diff: https://reviews.apache.org/r/35063/diff/ > > > Testing > ------- > > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Lens Checkstyle Rules ............................. SUCCESS [3.443s] > [INFO] Lens .............................................. SUCCESS [3.144s] > [INFO] Lens API .......................................... SUCCESS [19.933s] > [INFO] Lens API for server and extensions ................ SUCCESS [20.866s] > [INFO] Lens Cube ......................................... SUCCESS [3:30.712s] > [INFO] Lens DB storage ................................... SUCCESS [20.469s] > [INFO] Lens Query Library ................................ SUCCESS [16.834s] > [INFO] Lens Hive Driver .................................. SUCCESS [2:54.473s] > [INFO] Lens Driver for JDBC .............................. SUCCESS [37.130s] > [INFO] Lens Server ....................................... SUCCESS [5:42.657s] > [INFO] Lens client ....................................... SUCCESS [37.203s] > [INFO] Lens CLI .......................................... SUCCESS [4:10.969s] > [INFO] Lens Examples ..................................... SUCCESS [12.813s] > [INFO] Lens Distribution ................................. SUCCESS [8.697s] > [INFO] Lens ML Lib ....................................... SUCCESS [1:36.895s] > [INFO] Lens ML Ext Distribution .......................... SUCCESS [2.316s] > [INFO] Lens Regression ................................... SUCCESS [17.742s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 21:17.254s > [INFO] Finished at: Thu Jun 04 11:04:13 UTC 2015 > [INFO] Final Memory: 162M/1092M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Amareshwari Sriramadasu > >
