You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@hive.apache.org by Jaideep dhok <ja...@inmobi.com> on 2014/02/04 13:56:13 UTC

Review Request 17708: HIVE-6364 HiveServer2 - Request serving thread should get class loader from existing SessionState

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

Review request for hive and Thejas Nair.


Repository: hive-git


Description
-------

SessionState is created for each session in HS2. If we do any add jars, a class loader is set in the SessionState's conf object. This class loader should also be set in each thread that serves request of the same session.


Diffs
-----

  ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 64a8a60 
  ql/src/test/org/apache/hadoop/hive/ql/session/TestSessionState.java d4e737f 

Diff: https://reviews.apache.org/r/17708/diff/


Testing
-------

Added unit test that fails unless change in SessionState is applied.
Tested in local setup.


Thanks,

Jaideep dhok