hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Devaraj Das <d...@yahoo-inc.com>
Subject Re: Request for review
Date Fri, 01 Apr 2011 15:23:44 GMT
Sorry, didn't mean to send it on the dev list :-( .  Of course any other committer is welcome
to review :-)


On 4/1/11 8:20 AM, "Devaraj Das" <ddas@yahoo-inc.com> wrote:

Hi Amareshwari, could you please take a look at the patch for hive-1988. Since you are the
only hive committer at Y!, hence this request :-)
Devaraj


On 3/29/11 10:26 AM, "Devaraj Das" <ddas@apache.org> wrote:



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

 Review request for hive.
By Devaraj Das.
Updated 2011-03-29 10:26:38.374832
Description


Fixes to some security issues discussed in HIVE-1988

Testing (updated)


New unit test added and that passes. All unit tests passed.

 Bugs: HIVE-1988 <https://issues.apache.org/jira/browse/HIVE-1988>
Diffs

 *   http://svn.apache.org/repos/asf/hive/trunk/metastore/if/hive_metastore.thrift (1085623)
 *   http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
(1085623)
 *   http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
(1085623)
 *   http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java
(1085623)
 *   http://svn.apache.org/repos/asf/hive/trunk/shims/src/0.20S/java/org/apache/hadoop/hive/thrift/DelegationTokenSecretManager.java
(1085623)
 *   http://svn.apache.org/repos/asf/hive/trunk/shims/src/0.20S/java/org/apache/hadoop/hive/thrift/HadoopThriftAuthBridge20S.java
(1085623)
 *   http://svn.apache.org/repos/asf/hive/trunk/shims/src/common/java/org/apache/hadoop/hive/thrift/HadoopThriftAuthBridge.java
(1085623)
 *   http://svn.apache.org/repos/asf/hive/trunk/shims/src/test/org/apache/hadoop/hive/thrift/TestHadoop20SAuthBridge.java
(1085623)

View Diff <https://reviews.apache.org/r/528/diff/>




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