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

(Updated Feb. 14, 2016, 8:45 a.m.)


Review request for Sqoop and Colin Ma.


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


Repository: sqoop-sqoop2


Description
-------

Add admin user list to configuration file, and Sqoop can do the admin 
authorization according to the value.
For example:
 org.apache.sqoop.security.admins=admin1,admin2


Diffs (updated)
-----

  core/src/main/java/org/apache/sqoop/security/AuthorizationManager.java 
4afdf02 
  core/src/main/java/org/apache/sqoop/security/SecurityConstants.java 6f32e04 
  dist/src/main/conf/sqoop.properties 620146d 
  server/src/main/java/org/apache/sqoop/filter/SqoopAdminCheckFilter.java 
PRE-CREATION 
  server/src/main/java/org/apache/sqoop/server/SqoopJettyServer.java 2c4cb7a 

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


Testing
-------


Thanks,

Colin Ma

Reply via email to