----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10554/#review19455 -----------------------------------------------------------
Ship it! +1 - Ashutosh Chauhan On April 16, 2013, 9:46 p.m., Thejas Nair wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/10554/ > ----------------------------------------------------------- > > (Updated April 16, 2013, 9:46 p.m.) > > > Review request for hive. > > > Description > ------- > > remove duplicate impersonation parameters for hiveserver2 > > > This addresses bug HIVE-4356. > https://issues.apache.org/jira/browse/HIVE-4356 > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 78d9cc9 > conf/hive-default.xml.template e266ce7 > service/src/java/org/apache/hive/service/auth/PlainSaslHelper.java 18d4aae > service/src/java/org/apache/hive/service/cli/CLIService.java b53599b > service/src/java/org/apache/hive/service/cli/thrift/ThriftCLIService.java > 43d79aa > service/src/test/org/apache/hive/service/auth/TestPlainSaslHelper.java > PRE-CREATION > > service/src/test/org/apache/hive/service/cli/thrift/TestThriftCLIService.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/10554/diff/ > > > Testing > ------- > > Unit tests included. > Manually tested on (kerberos) secure and unsecure cluster. > > > Thanks, > > Thejas Nair > >
