----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19206/#review37222 -----------------------------------------------------------
Ship it! LGTM pending tests - Brock Noland On March 14, 2014, 5:12 p.m., Prasad Mujumdar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19206/ > ----------------------------------------------------------- > > (Updated March 14, 2014, 5:12 p.m.) > > > Review request for hive, Brock Noland and Thejas Nair. > > > Bugs: HIVE-6657 > https://issues.apache.org/jira/browse/HIVE-6657 > > > Repository: hive-git > > > Description > ------- > > - Build changes > + pull required dependenices > + new sub-module under itest for the hive miniKdc framework, enabled for > hadoop-2 only since hadoop-1 doesn't include miniKdc yet. > - A wrapper class MiniHiveKdc on top of Hadoop miniKdc that manages the kdc > services that creates test principals, keytab etc > - Tests > + simple unit test for MiniHiveKdc > + Integrate MiniHiveKdc with MiniHS2 > + New HiveServer2 test that covers basic connection, as well as the token > and proxy auth support added by HIVE-5155 > > Note that Hadoop does provide a secure mini-cluster yet. Hence the new tests > are not executing any queries. At this point it's limited to authentication > part only. > > > Diffs > ----- > > itests/hive-minikdc/pom.xml PRE-CREATION > itests/hive-minikdc/src/test/java/org/apache/hive/minikdc/MiniHiveKdc.java > PRE-CREATION > > itests/hive-minikdc/src/test/java/org/apache/hive/minikdc/TestJdbcWithMiniKdc.java > PRE-CREATION > > itests/hive-minikdc/src/test/java/org/apache/hive/minikdc/TestMiniHiveKdc.java > PRE-CREATION > itests/hive-minikdc/src/test/resources/core-site.xml PRE-CREATION > itests/hive-unit/pom.xml 6aff277 > > itests/hive-unit/src/main/java/org/apache/hive/jdbc/miniHS2/AbstractHiveService.java > PRE-CREATION > itests/hive-unit/src/main/java/org/apache/hive/jdbc/miniHS2/MiniHS2.java > PRE-CREATION > > itests/hive-unit/src/test/java/org/apache/hive/jdbc/miniHS2/AbstractHiveService.java > 095b989 > itests/hive-unit/src/test/java/org/apache/hive/jdbc/miniHS2/MiniHS2.java > d76750c > > itests/hive-unit/src/test/java/org/apache/hive/jdbc/miniHS2/TestHiveServer2.java > 84f508c > itests/pom.xml 589ba25 > pom.xml 6503c94 > service/src/java/org/apache/hive/service/auth/HiveAuthFactory.java 6759903 > > service/src/java/org/apache/hive/service/cli/thrift/ThriftBinaryCLIService.java > 6fbc847 > > Diff: https://reviews.apache.org/r/19206/diff/ > > > Testing > ------- > > New test cases using the miniKdc. > > > Thanks, > > Prasad Mujumdar > >