----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/586/#review459 -----------------------------------------------------------
Ship it! +1 Please upload the patch to jira. - Amareshwari On 2011-04-14 00:57:35, Devaraj Das wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/586/ > ----------------------------------------------------------- > > (Updated 2011-04-14 00:57:35) > > > Review request for hive. > > > Summary > ------- > > This is a patch to handle the issue described in HIVE-1943 > > > This addresses bug HIVE-1943. > https://issues.apache.org/jira/browse/HIVE-1943 > > > Diffs > ----- > > > http://svn.apache.org/repos/asf/hive/trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java > 1091578 > http://svn.apache.org/repos/asf/hive/trunk/conf/hive-default.xml 1091578 > > http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java > 1091578 > > http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java > 1091578 > > http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/Warehouse.java > 1091578 > > http://svn.apache.org/repos/asf/hive/trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestMetaStoreAuthorization.java > PRE-CREATION > > http://svn.apache.org/repos/asf/hive/trunk/shims/src/0.20/java/org/apache/hadoop/hive/shims/Hadoop20Shims.java > 1091578 > > http://svn.apache.org/repos/asf/hive/trunk/shims/src/0.20S/java/org/apache/hadoop/hive/shims/Hadoop20SShims.java > 1091578 > > http://svn.apache.org/repos/asf/hive/trunk/shims/src/common/java/org/apache/hadoop/hive/shims/HadoopShims.java > 1091578 > > http://svn.apache.org/repos/asf/hive/trunk/shims/src/test/org/apache/hadoop/hive/thrift/TestHadoop20SAuthBridge.java > 1091578 > > Diff: https://reviews.apache.org/r/586/diff > > > Testing > ------- > > Unit tests added. > > > Thanks, > > Devaraj > >
