-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31589/
-----------------------------------------------------------
(Updated March 1, 2015, 8:27 a.m.)
Review request for lens and Amareshwari Sriramadasu.
Changes
-------
Removed 'conf' instance variable in ColumnarSQLRewriter
Bugs: LENS-370
https://issues.apache.org/jira/browse/LENS-370
Repository: lens
Description
-------
Setting queryConf variable in rewriter which is set after every refresh.
Copy class loader of driver into queryConf
Diffs (updated)
-----
lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/ColumnarSQLRewriter.java
db653a7154f64fa1343698e501d69dc9dc277950
lens-driver-jdbc/src/main/java/org/apache/lens/driver/jdbc/JDBCDriver.java
83b956e355d25280a187fe7190300fee757a315d
lens-driver-jdbc/src/test/java/org/apache/lens/driver/jdbc/TestColumnarSQLRewriter.java
c06196054cb550ee89796a31267c60d86187cbb5
lens-driver-jdbc/testdata/ClassLoaderTestClass.java PRE-CREATION
lens-driver-jdbc/testdata/ClassLoaderTestClass2.java PRE-CREATION
lens-driver-jdbc/testdata/DatabaseJarSerde.java PRE-CREATION
lens-driver-jdbc/testdata/serde.jar PRE-CREATION
lens-driver-jdbc/testdata/test.jar PRE-CREATION
lens-driver-jdbc/testdata/test2.jar PRE-CREATION
lens-driver-jdbc/testdata/testdata2.data PRE-CREATION
Diff: https://reviews.apache.org/r/31589/diff/
Testing
-------
Changed existing unit test to use class with serde in a jar which is added at
runtime.
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Lens Checkstyle Rules ............................. SUCCESS [2.145s]
[INFO] Lens .............................................. SUCCESS [1.879s]
[INFO] Lens API .......................................... SUCCESS [5.367s]
[INFO] Lens API for server and extensions ................ SUCCESS [6.541s]
[INFO] Lens Cube ......................................... SUCCESS [6:00.834s]
[INFO] Lens DB storage ................................... SUCCESS [10.603s]
[INFO] Lens Query Library ................................ SUCCESS [4.311s]
[INFO] Lens Hive Driver .................................. SUCCESS [2:34.355s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [27.769s]
[INFO] Lens Server ....................................... SUCCESS [4:31.677s]
[INFO] Lens client ....................................... SUCCESS [20.716s]
[INFO] Lens CLI .......................................... SUCCESS [1:52.615s]
[INFO] Lens Examples ..................................... SUCCESS [0.794s]
[INFO] Lens Distribution ................................. SUCCESS [9.913s]
[INFO] Lens ML Lib ....................................... SUCCESS [42.423s]
[INFO] Lens Regression ................................... SUCCESS [0.484s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 17:13.361s
[INFO] Finished at: Sun Mar 01 08:10:25 UTC 2015
[INFO] Final Memory: 100M/1017M
[INFO] ------------------------------------------------------------------------
Thanks,
Jaideep dhok
