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



common/src/java/org/apache/hadoop/hive/conf/HiveConf.java
<https://reviews.apache.org/r/12824/#comment47598>

    should we just call this 
    hive.server2.thrift.sasl.qop ? That seems more self describing.
    


- Thejas Nair


On July 22, 2013, 8:56 p.m., Arup Malakar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12824/
> -----------------------------------------------------------
> 
> (Updated July 22, 2013, 8:56 p.m.)
> 
> 
> Review request for hive.
> 
> 
> 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
> -----
> 
>   common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 
> 11c31216495d0c4e454f2627af5c93a9f270b1fe 
>   data/conf/hive-site.xml 4e6ff16135833da1a4df12a12a6fe59ad4f870ba 
>   jdbc/src/java/org/apache/hive/jdbc/HiveConnection.java 
> 00f43511b478c687b7811fc8ad66af2b507a3626 
>   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/java/org/apache/hadoop/hive/thrift/HadoopThriftAuthBridge.java
>  9b0ec0a75563b41339e6fc747556440fdf83e31e 
> 
> Diff: https://reviews.apache.org/r/12824/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Arup Malakar
> 
>

Reply via email to