hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Prasad Mujumdar" <pras...@cloudera.com>
Subject Re: Review Request 11326: HIVE-4588: Support session level hooks for HiveServer2
Date Thu, 22 Aug 2013 18:54:59 GMT

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

(Updated Aug. 22, 2013, 6:54 p.m.)


Review request for hive and Navis Ryu.


Changes
-------

Added a method in Driver.java to log error messages related to loading hook classes. 
Some unrelated code cleanup to use interface for semantic hook instead of abstract class.


Bugs: HIVE-4588
    https://issues.apache.org/jira/browse/HIVE-4588


Repository: hive-git


Description
-------

Support session level hooks for HiveServer2
  - New config parameter to define the hook
  - New hook context interface to pass the serssion user and config to the hook implementation
  - Session manager executes the configured hooks when a new session starts


Diffs (updated)
-----

  common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 0b4dad9 
  ql/src/java/org/apache/hadoop/hive/ql/Driver.java 3312b3b 
  ql/src/java/org/apache/hadoop/hive/ql/hooks/HookUtils.java PRE-CREATION 
  service/src/java/org/apache/hive/service/cli/session/HiveSessionHook.java PRE-CREATION 
  service/src/java/org/apache/hive/service/cli/session/HiveSessionHookContext.java PRE-CREATION

  service/src/java/org/apache/hive/service/cli/session/HiveSessionHookContextImpl.java PRE-CREATION

  service/src/java/org/apache/hive/service/cli/session/SessionManager.java 3bb6807 
  service/src/test/org/apache/hive/service/cli/session/TestSessionHooks.java PRE-CREATION


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


Testing
-------

Added new test for session hooks


Thanks,

Prasad Mujumdar


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message