----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32427/ -----------------------------------------------------------
(Updated March 26, 2015, 2:18 a.m.) Review request for Sqoop. Changes ------- Removed some function by mistake Repository: sqoop-sqoop2 Description ------- When getting multi-connectors/links/jobs, AuthorizationEngine.filterResource is used to filter resources according to the permissions. When getting one specific connector/link/job via id, this resource should be checked permission, and throw exception if no privilege. Diffs (updated) ----- common/src/main/java/org/apache/sqoop/error/code/CommonRepositoryError.java 7db31ddbe7332ac4053495328dc7e932e2dad66c core/src/main/java/org/apache/sqoop/connector/ConnectorManager.java 0b150468ee10b7b455238b9e1972cb635ef7d6f2 security/src/main/java/org/apache/sqoop/security/Authorization/AuthorizationEngine.java d261027ff5aefa728c5f761534f4439264b4ae45 server/src/main/java/org/apache/sqoop/handler/ConnectorRequestHandler.java 2305a51472a0201771fe493e8364f7db62ee79a2 server/src/main/java/org/apache/sqoop/handler/JobRequestHandler.java 0c5f1f216e9e6a42bd24c226dbca18cacd4f5c55 server/src/main/java/org/apache/sqoop/handler/LinkRequestHandler.java 3187bcf00b51533cc33513a315dd9f1296e58c6d Diff: https://reviews.apache.org/r/32427/diff/ Testing ------- manual Thanks, richard zhou
