-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32478/
-----------------------------------------------------------
(Updated March 27, 2015, 10:41 a.m.)
Review request for lens and Amareshwari Sriramadasu.
Bugs: LENS-349
https://issues.apache.org/jira/browse/LENS-349
Repository: lens
Description
-------
1. Set database in AbstractQueryContext at construction time. This will make
sure that database used at the time of query submission is also used at the
time of query execution
2. HiveDriver will maintain one session for a lens session + database name pair.
3. Borrowed some code related to closing class loaders and tests from the
previous patch.
Reply
Diffs (updated)
-----
lens-client/src/test/java/org/apache/lens/client/TestLensClient.java
81a536ed8ac3f7ea550794cf188db6ea9d599c7d
lens-driver-hive/src/main/java/org/apache/lens/driver/hive/HiveDriver.java
9e3c72393c1d9634b4ae03301b5d620b647df7ae
lens-driver-hive/src/test/java/org/apache/lens/driver/hive/TestHiveDriver.java
8a5839b01ad20fc6ff030d55788ceb2c368d7f69
lens-driver-jdbc/testdata/DatabaseJarSerde.java
03caff31916072b0e6944d1bd13161dca5cef878
lens-server-api/src/main/java/org/apache/lens/server/api/query/AbstractQueryContext.java
523356903933af37eb9b1deca805cc8ae76a4cbb
lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java
390071cdd40ccc0308a4a58852e9dbfcbddabc3a
lens-server/src/main/java/org/apache/lens/server/session/LensSessionImpl.java
8c970821015b04e4adaff7d2949520ea5a0c4c82
lens-server/src/test/java/org/apache/lens/server/LensJerseyTest.java
20856a09610d76ca8072851e00125fc50aa4343c
lens-server/src/test/java/org/apache/lens/server/LensTestUtil.java
e44816372e889bdc7ce2ac34a43048e8af85181e
lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java
14a4eb2f09ad07bd6592f9f2e7ebff02080e0d14
lens-server/testdata/DatabaseJarSerde.java
03caff31916072b0e6944d1bd13161dca5cef878
lens-server/testdata/serde.jar ec86e49a0be7cb9872756a4313ae81bd3cb5e543
lens-server/testdata/test.jar 1644d8cada37749f6a8c3a2a6c26b752ea7bac0f
Diff: https://reviews.apache.org/r/32478/diff/
Testing
-------
Changed existing test for db jars to validate db switch
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Lens Checkstyle Rules ............................. SUCCESS [2.087s]
[INFO] Lens .............................................. SUCCESS [1.810s]
[INFO] Lens API .......................................... SUCCESS [5.499s]
[INFO] Lens API for server and extensions ................ SUCCESS [6.904s]
[INFO] Lens Cube ......................................... SUCCESS [2:17.874s]
[INFO] Lens DB storage ................................... SUCCESS [15.951s]
[INFO] Lens Query Library ................................ SUCCESS [6.794s]
[INFO] Lens Hive Driver .................................. SUCCESS [2:39.982s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [17.581s]
[INFO] Lens Server ....................................... SUCCESS [5:29.163s]
[INFO] Lens client ....................................... SUCCESS [20.579s]
[INFO] Lens CLI .......................................... SUCCESS [2:41.566s]
[INFO] Lens Examples ..................................... SUCCESS [0.809s]
[INFO] Lens Distribution ................................. SUCCESS [10.192s]
[INFO] Lens ML Lib ....................................... SUCCESS [1:03.257s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [2.610s]
[INFO] Lens Regression ................................... SUCCESS [0.461s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 15:44.150s
[INFO] Finished at: Fri Mar 27 10:35:15 UTC 2015
[INFO] Final Memory: 107M/1280M
[INFO] ------------------------------------------------------------------------
Thanks,
Jaideep dhok
