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

(Updated 2011-12-20 02:05:22.736212)


Review request for hive.


Changes
-------

Updated patch incorporating Carl's comments.
Also added tests as requested by Carl.


Summary
-------

Pass user identity in metastore connection in unsecure mode


This addresses bug HIVE-2616.
    https://issues.apache.org/jira/browse/HIVE-2616


Diffs (updated)
-----

  trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 1221059 
  trunk/conf/hive-default.xml.template 1221059 
  trunk/metastore/if/hive_metastore.thrift 1221059 
  trunk/metastore/src/gen/thrift/gen-cpp/ThriftHiveMetastore.h 1221059 
  trunk/metastore/src/gen/thrift/gen-cpp/ThriftHiveMetastore.cpp 1221059 
  
trunk/metastore/src/gen/thrift/gen-cpp/ThriftHiveMetastore_server.skeleton.cpp 
1221059 
  
trunk/metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/ThriftHiveMetastore.java
 1221059 
  trunk/metastore/src/gen/thrift/gen-php/hive_metastore/ThriftHiveMetastore.php 
1221059 
  
trunk/metastore/src/gen/thrift/gen-py/hive_metastore/ThriftHiveMetastore-remote 
1221059 
  trunk/metastore/src/gen/thrift/gen-py/hive_metastore/ThriftHiveMetastore.py 
1221059 
  trunk/metastore/src/gen/thrift/gen-rb/thrift_hive_metastore.rb 1221059 
  trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java 
1221059 
  
trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
 1221059 
  
trunk/metastore/src/java/org/apache/hadoop/hive/metastore/TUGIBasedProcessor.java
 PRE-CREATION 
  
trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestRemoteHiveMetaStore.java
 1221059 
  
trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestSetUGIOnBothClientServer.java
 PRE-CREATION 
  
trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestSetUGIOnOnlyClient.java
 PRE-CREATION 
  
trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestSetUGIOnOnlyServer.java
 PRE-CREATION 
  trunk/shims/ivy.xml 1221059 
  trunk/shims/src/0.20/java/org/apache/hadoop/hive/shims/Hadoop20Shims.java 
1221059 
  trunk/shims/src/0.20S/java/org/apache/hadoop/hive/shims/Hadoop20SShims.java 
1221059 
  
trunk/shims/src/0.20S/java/org/apache/hadoop/hive/thrift/HadoopThriftAuthBridge20S.java
 1221059 
  
trunk/shims/src/0.20S/java/org/apache/hadoop/hive/thrift/client/TUGIAssumingTransport.java
 PRE-CREATION 
  trunk/shims/src/common/java/org/apache/hadoop/hive/shims/HadoopShims.java 
1221059 
  
trunk/shims/src/common/java/org/apache/hadoop/hive/thrift/TFilterTransport.java 
PRE-CREATION 
  
trunk/shims/src/common/java/org/apache/hadoop/hive/thrift/TUGIContainingTransport.java
 PRE-CREATION 

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


Testing
-------

All the tests in metastore dir passes. Manually tested that file on hdfs is 
owned by user running the client and not by user running metastore server.


Thanks,

Ashutosh

Reply via email to