-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/12824/
-----------------------------------------------------------
(Updated Aug. 2, 2013, 10:51 p.m.)
Review request for hive.
Changes
-------
1. Incorporated sasl.qop renaming of param
2. Moved getHadoopSaslProperties to HadoopThriftAuthBridge
Bugs: HIVE-4911
https://issues.apache.org/jira/browse/HIVE-4911
Repository: hive-git
Description
-------
The QoP for hive server 2 should be configurable to enable encryption. A new
configuration should be exposed "hive.server2.thrift.rpc.protection". This
would give greater control configuring hive server 2 service.
Diffs (updated)
-----
common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
11c31216495d0c4e454f2627af5c93a9f270b1fe
conf/hive-default.xml.template 603b475802152a4bd5ab92a4c7146b56f6be020d
jdbc/src/java/org/apache/hive/jdbc/HiveConnection.java
00f43511b478c687b7811fc8ad66af2b507a3626
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
72eac989394a3899998e52d3845b02bb38ebeaad
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
cef50f40ccb047a8135f704b2997968a2cf477b8
metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
88151a1d48b12cf3a8346ae94b6d1a182a331992
service/src/java/org/apache/hive/service/auth/HiveAuthFactory.java
1809e1b26ceee5de14a354a0e499aa8c0ab793bf
service/src/java/org/apache/hive/service/auth/KerberosSaslHelper.java
379dafb8377aed55e74f0ae18407996bb9e1216f
service/src/java/org/apache/hive/service/auth/SaslQOP.java PRE-CREATION
shims/src/common-secure/java/org/apache/hadoop/hive/thrift/HadoopThriftAuthBridge20S.java
777226f8da0af2235d4294cd6a676fa8192c89e4
shims/src/common-secure/test/org/apache/hadoop/hive/thrift/TestHadoop20SAuthBridge.java
172e03115372dc2c742469cbc5f0fefd1053163d
shims/src/common/java/org/apache/hadoop/hive/thrift/HadoopThriftAuthBridge.java
9b0ec0a75563b41339e6fc747556440fdf83e31e
Diff: https://reviews.apache.org/r/12824/diff/
Testing
-------
Thanks,
Arup Malakar
