----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30790/#review72121 -----------------------------------------------------------
Ship it! LGTM server/src/main/webapp/WEB-INF/web.xml <https://reviews.apache.org/r/30790/#comment118101> /v1/authorization/* - Abraham Elmahrek On Feb. 10, 2015, 7:39 a.m., richard zhou wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30790/ > ----------------------------------------------------------- > > (Updated Feb. 10, 2015, 7:39 a.m.) > > > Review request for Sqoop and shen guoquan. > > > Repository: sqoop-sqoop2 > > > Description > ------- > > Handle Rest API in server side to grant/revoke role, etc. > > > Diffs > ----- > > common/src/main/java/org/apache/sqoop/json/PrincipalBean.java PRE-CREATION > common/src/main/java/org/apache/sqoop/json/PrincipalsBean.java PRE-CREATION > common/src/main/java/org/apache/sqoop/json/PrivilegeBean.java PRE-CREATION > common/src/main/java/org/apache/sqoop/json/PrivilegesBean.java PRE-CREATION > common/src/main/java/org/apache/sqoop/json/RoleBean.java PRE-CREATION > common/src/main/java/org/apache/sqoop/json/RolesBean.java PRE-CREATION > core/src/main/java/org/apache/sqoop/security/SecurityError.java > 322a98f1f148d7544c7fe5e9a84ef8e9b6618612 > > server/src/main/java/org/apache/sqoop/handler/AuthorizationRequestHandler.java > PRE-CREATION > server/src/main/java/org/apache/sqoop/server/AuthorizationServlet.java > PRE-CREATION > server/src/main/java/org/apache/sqoop/server/RequestHandler.java > bd62f31eac3e47c2eb085f836383f98b17260d2f > server/src/main/webapp/WEB-INF/web.xml > 85ae26ba0839dd897527c9fe13e4e56c84d43ecc > > Diff: https://reviews.apache.org/r/30790/diff/ > > > Testing > ------- > > local > > > Thanks, > > richard zhou > >
