> On Nov. 25, 2015, 1:10 p.m., Amareshwari Sriramadasu wrote: > > lens-cube/src/main/java/org/apache/lens/cube/metadata/CubeMetastoreClient.java, > > line 1496 > > <https://reviews.apache.org/r/40597/diff/5/?file=1146571#file1146571line1496> > > > > Why are we removing dimTable != null check ? I just checked > > getDimensionTable(tbl) method, it can return null.
No, it's has only one line: `return new CubeDimensionTable(tbl);` > On Nov. 25, 2015, 1:10 p.m., Amareshwari Sriramadasu wrote: > > lens-cube/src/main/java/org/apache/lens/cube/metadata/CubeMetastoreClient.java, > > line 1080 > > <https://reviews.apache.org/r/40597/diff/5/?file=1146571#file1146571line1080> > > > > Should name of this method be getLatestTimestampFromPartition ? Renamed - Rajat ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40597/#review107935 ----------------------------------------------------------- On Nov. 25, 2015, 1:39 p.m., Rajat Khandelwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40597/ > ----------------------------------------------------------- > > (Updated Nov. 25, 2015, 1:39 p.m.) > > > Review request for lens. > > > Bugs: LENS-871 > https://issues.apache.org/jira/browse/LENS-871 > > > Repository: lens > > > Description > ------- > > > Diffs > ----- > > > lens-cube/src/main/java/org/apache/lens/cube/metadata/CubeMetastoreClient.java > 1f13617be4f964c8ed7326f57bd404ee354aa77f > lens-cube/src/main/java/org/apache/lens/cube/metadata/MetastoreUtil.java > e5cf468db36bb9e29d79bd4ca1d11a9f32c74e53 > lens-cube/src/main/java/org/apache/lens/cube/metadata/StorageTableDesc.java > 0a2c5dfdcf819690c7fa7573a955791c84a19d5e > lens-cube/src/main/java/org/apache/lens/cube/metadata/UpdatePeriod.java > f192463dc2d32d9a783495ef299869a56281cc15 > lens-cube/src/main/java/org/apache/lens/cube/parse/DateUtil.java > 4690d1d479e78dcd5ec46a009d1d64734c4302e9 > > lens-cube/src/test/java/org/apache/lens/cube/metadata/TestCubeMetastoreClient.java > 6b6f645ed9105fa535a63c12abdfa3f80d8a734c > lens-cube/src/test/java/org/apache/lens/cube/parse/CubeTestSetup.java > 999faa07a0ef4cecd7b765156268f0ba7c0d8e60 > lens-server-api/src/main/java/org/apache/lens/server/api/util/LensUtil.java > 6dcb2d79420b29fa85bd115f375c0c7e42c50834 > > lens-server/src/main/java/org/apache/lens/server/metastore/CubeMetastoreServiceImpl.java > 27f9e0e016db83a38865e8fdb02188b65a0df531 > > lens-server/src/test/java/org/apache/lens/server/metastore/TestMetastoreService.java > 5f9f6396954f56fa13914e546ece597c7fdeaa88 > > Diff: https://reviews.apache.org/r/40597/diff/ > > > Testing > ------- > > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Lens Checkstyle Rules ............................. SUCCESS [1.814s] > [INFO] Lens .............................................. SUCCESS [2.908s] > [INFO] Lens API .......................................... SUCCESS [26.054s] > [INFO] Lens API for server and extensions ................ SUCCESS [21.785s] > [INFO] Lens Cube ......................................... SUCCESS [5:37.766s] > [INFO] Lens DB storage ................................... SUCCESS [20.543s] > [INFO] Lens Query Library ................................ SUCCESS [17.053s] > [INFO] Lens Hive Driver .................................. SUCCESS [2:49.187s] > [INFO] Lens Driver for JDBC .............................. SUCCESS [36.206s] > [INFO] Lens Elastic Search Driver ........................ SUCCESS [17.148s] > [INFO] Lens Server ....................................... SUCCESS [7:52.960s] > [INFO] Lens client ....................................... SUCCESS [34.571s] > [INFO] Lens CLI .......................................... SUCCESS [52.979s] > [INFO] Lens Examples ..................................... SUCCESS [10.169s] > [INFO] Lens Ship Jars to Distributed Cache ............... SUCCESS [1.466s] > [INFO] Lens Distribution ................................. SUCCESS [9.277s] > [INFO] Lens ML Lib ....................................... SUCCESS [1:17.322s] > [INFO] Lens ML Ext Distribution .......................... SUCCESS [1.895s] > [INFO] Lens Regression ................................... SUCCESS [12.466s] > [INFO] Lens UI ........................................... SUCCESS [26.585s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 22:31.093s > [INFO] Finished at: Tue Nov 24 12:37:46 UTC 2015 > [INFO] Final Memory: 194M/1512M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Rajat Khandelwal > >
