-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/32478/
-----------------------------------------------------------
(Updated March 30, 2015, 8:06 a.m.)
Review request for lens and Amareshwari Sriramadasu.
Changes
-------
rebased with master and review comments by Himanshu
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
218dc53145398f1fc584efa6160c49fb6b8f2d0c
lens-driver-hive/src/test/java/org/apache/lens/driver/hive/TestHiveDriver.java
b16c34665c9cceeb40cbd0d633a5f8d2802de381
lens-driver-jdbc/testdata/DatabaseJarSerde.java
03caff31916072b0e6944d1bd13161dca5cef878
lens-server-api/src/main/java/org/apache/lens/server/api/query/AbstractQueryContext.java
6799e0c7a0f487280a581b9ba034f70a1c814418
lens-server/src/main/java/org/apache/lens/server/query/QueryExecutionServiceImpl.java
ea2da14f51dc0eb4f0e06e73ae7752f1fe5cb769
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/TestServerRestart.java
6306b51cdf2607c9762aec522d663019b202760f
lens-server/src/test/java/org/apache/lens/server/query/TestQueryService.java
e3e3d4baf43f67f96687b47b3273046e24720254
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 (updated)
-------
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Lens Checkstyle Rules ............................. SUCCESS [23.993s]
[INFO] Lens .............................................. SUCCESS [24.084s]
[INFO] Lens API .......................................... SUCCESS [13.620s]
[INFO] Lens API for server and extensions ................ SUCCESS [21.656s]
[INFO] Lens Cube ......................................... SUCCESS [3:16.315s]
[INFO] Lens DB storage ................................... SUCCESS [17.831s]
[INFO] Lens Query Library ................................ SUCCESS [10.164s]
[INFO] Lens Hive Driver .................................. SUCCESS [4:12.615s]
[INFO] Lens Driver for JDBC .............................. SUCCESS [27.693s]
[INFO] Lens Server ....................................... SUCCESS [19:58.109s]
[INFO] Lens client ....................................... SUCCESS [31.580s]
[INFO] Lens CLI .......................................... SUCCESS [6:33.093s]
[INFO] Lens Examples ..................................... SUCCESS [3.288s]
[INFO] Lens Distribution ................................. SUCCESS [18.011s]
[INFO] Lens ML Lib ....................................... SUCCESS [2:30.753s]
[INFO] Lens ML Ext Distribution .......................... SUCCESS [10.808s]
[INFO] Lens Regression ................................... SUCCESS [0.643s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 39:55.049s
[INFO] Finished at: Mon Mar 30 13:33:40 IST 2015
[INFO] Final Memory: 128M/840M
[INFO] ------------------------------------------------------------------------
Thanks,
Jaideep dhok
