sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jarek Cecho" <jar...@apache.org>
Subject Review Request 40625: SQOOP-2709 Sqoop2: HDFS: Impersonation on secured cluster doesn't work
Date Tue, 24 Nov 2015 00:58:55 GMT

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

Review request for Sqoop.


Bugs: SQOOP-2709
    https://issues.apache.org/jira/browse/SQOOP-2709


Repository: sqoop-sqoop2


Description
-------

I've provided util class that can retrieve delegation token for "current" user and store it
in our Context that is passed to execution engine.


Diffs
-----

  connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsConstants.java
39ee4a3 
  connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsExtractor.java
583acdd 
  connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsFromInitializer.java
be837ca 
  connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsLoader.java 04acd18

  connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsPartitioner.java
998b903 
  connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsToDestroyer.java
2bad23a 
  connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsToInitializer.java
5856371 
  connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/security/SecurityUtils.java
PRE-CREATION 
  connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/security/TestSecurityUtils.java
PRE-CREATION 

Diff: https://reviews.apache.org/r/40625/diff/


Testing
-------

I've tested the patch on secured real cluster to make sure that it's working. Sadly I did
not included any integration test as our suite currently doesn't have any support for MiniKDC
(this is something that we will add later).


Thanks,

Jarek Cecho


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