----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39680/#review104152 -----------------------------------------------------------
lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/ColumnarSQLRewriter.java (line 180) <https://reviews.apache.org/r/39680/#comment162426> We can remove new HashMap<>() lens-driver-jdbc/src/main/resources/jdbcdriver-default.xml (line 33) <https://reviews.apache.org/r/39680/#comment162425> Move to tests? Can you include unit test? - Amareshwari Sriramadasu On Oct. 27, 2015, 8:08 a.m., Rajat Khandelwal wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39680/ > ----------------------------------------------------------- > > (Updated Oct. 27, 2015, 8:08 a.m.) > > > Review request for lens. > > > Bugs: LENS-848 > https://issues.apache.org/jira/browse/LENS-848 > > > Repository: lens > > > Description > ------- > > Will plan to separate properties later. Keeping them for backward > compatibility. > > > Diffs > ----- > > lens-api/src/main/java/org/apache/lens/api/util/CommonUtils.java > PRE-CREATION > > lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/ColumnarSQLRewriter.java > ccb2b7fdf1fe297b7240ce1efcb1646303e1236d > > lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/DataSourceConnectionProvider.java > b14835a3c40eabd9b0939e9be19e41852e5f99c4 > > lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/JDBCDriverConfConstants.java > 70815bbb8d89685fd2f3faf14230b8b458270ff1 > lens-driver-jdbc/src/main/resources/jdbcdriver-default.xml > 163ed42daf49a182ead3eca048b3ada6e53e2c0b > > Diff: https://reviews.apache.org/r/39680/diff/ > > > Testing > ------- > > [INFO] > ------------------------------------------------------------------------ > [INFO] Reactor Summary: > [INFO] > [INFO] Lens Checkstyle Rules ............................. SUCCESS [2.414s] > [INFO] Lens .............................................. SUCCESS [3.298s] > [INFO] Lens API .......................................... SUCCESS [27.005s] > [INFO] Lens API for server and extensions ................ SUCCESS [25.469s] > [INFO] Lens Cube ......................................... SUCCESS [5:46.494s] > [INFO] Lens DB storage ................................... SUCCESS [22.510s] > [INFO] Lens Query Library ................................ SUCCESS [19.621s] > [INFO] Lens Hive Driver .................................. SUCCESS [2:57.116s] > [INFO] Lens Driver for JDBC .............................. SUCCESS [40.236s] > [INFO] Lens Elastic Search Driver ........................ SUCCESS [19.414s] > [INFO] Lens Server ....................................... SUCCESS [8:21.803s] > [INFO] Lens client ....................................... SUCCESS [36.730s] > [INFO] Lens CLI .......................................... SUCCESS [54.984s] > [INFO] Lens Examples ..................................... SUCCESS [10.850s] > [INFO] Lens Ship Jars to Distributed Cache ............... SUCCESS [1.867s] > [INFO] Lens Distribution ................................. SUCCESS [9.404s] > [INFO] Lens ML Lib ....................................... SUCCESS [1:24.110s] > [INFO] Lens ML Ext Distribution .......................... SUCCESS [1.979s] > [INFO] Lens Regression ................................... SUCCESS [13.230s] > [INFO] Lens UI ........................................... SUCCESS [27.486s] > [INFO] > ------------------------------------------------------------------------ > [INFO] BUILD SUCCESS > [INFO] > ------------------------------------------------------------------------ > [INFO] Total time: 23:47.270s > [INFO] Finished at: Tue Oct 27 08:04:55 UTC 2015 > [INFO] Final Memory: 179M/1101M > [INFO] > ------------------------------------------------------------------------ > > > Thanks, > > Rajat Khandelwal > >
