hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Prasad Mujumdar" <pras...@cloudera.com>
Subject Re: Review Request 19206: HIVE-6657: Add test coverage for Kerberos authentication implementation using Hadoop's miniKdc
Date Fri, 14 Mar 2014 07:06:52 GMT

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

(Updated March 14, 2014, 7:06 a.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


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