-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/31589/#review74697
-----------------------------------------------------------

Ship it!


Ship It!

- Rajat Khandelwal


On March 1, 2015, 2:36 p.m., Jaideep dhok wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/31589/
> -----------------------------------------------------------
> 
> (Updated March 1, 2015, 2:36 p.m.)
> 
> 
> Review request for lens and Amareshwari Sriramadasu.
> 
> 
> 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
> -----
> 
>   
> 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.
> Test output after all review changes.
> 
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Reactor Summary:
> [INFO] 
> [INFO] Lens Checkstyle Rules ............................. SUCCESS [2.127s]
> [INFO] Lens .............................................. SUCCESS [1.850s]
> [INFO] Lens API .......................................... SUCCESS [5.819s]
> [INFO] Lens API for server and extensions ................ SUCCESS [5.334s]
> [INFO] Lens Cube ......................................... SUCCESS [6:08.328s]
> [INFO] Lens DB storage ................................... SUCCESS [9.987s]
> [INFO] Lens Query Library ................................ SUCCESS [4.624s]
> [INFO] Lens Hive Driver .................................. SUCCESS [2:34.865s]
> [INFO] Lens Driver for JDBC .............................. SUCCESS [25.450s]
> [INFO] Lens Server ....................................... SUCCESS [4:26.486s]
> [INFO] Lens client ....................................... SUCCESS [20.688s]
> [INFO] Lens CLI .......................................... SUCCESS [1:52.629s]
> [INFO] Lens Examples ..................................... SUCCESS [0.969s]
> [INFO] Lens Distribution ................................. SUCCESS [10.075s]
> [INFO] Lens ML Lib ....................................... SUCCESS [42.667s]
> [INFO] Lens Regression ................................... SUCCESS [0.493s]
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] BUILD SUCCESS
> [INFO] 
> ------------------------------------------------------------------------
> [INFO] Total time: 17:13.309s
> [INFO] Finished at: Sun Mar 01 09:00:44 UTC 2015
> [INFO] Final Memory: 100M/1046M
> [INFO] 
> ------------------------------------------------------------------------
> 
> 
> Thanks,
> 
> Jaideep dhok
> 
>

Reply via email to