----------------------------------------------------------- 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