----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73354/#review223374 -----------------------------------------------------------
Ship it! Ship It! - Sailaja Polavarapu On Aug. 19, 2021, 4:24 a.m., Abhishek Kumar wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73354/ > ----------------------------------------------------------- > > (Updated Aug. 19, 2021, 4:24 a.m.) > > > Review request for ranger, Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, > Pradeep Agrawal, Sailaja Polavarapu, and Velmurugan Periasamy. > > > Bugs: RANGER-3285 > https://issues.apache.org/jira/browse/RANGER-3285 > > > Repository: ranger > > > Description > ------- > > New features implemented in this patch: > 1. A new coulmn 'sync_source' is available for db tables x_user, > x_portal_user and x_group. This column is also a search field for REST API > queries. > 2. Tested the functionality on containers for proper functioning. > 3. Front-End side changes to follow soon. > > > Diffs > ----- > > security-admin/db/mysql/optimized/current/ranger_core_db_mysql.sql > 1ba3e69e8 > > security-admin/db/mysql/patches/055-add-syncSource-col-in-x_user-x_portal_user-x_group.sql > PRE-CREATION > security-admin/db/oracle/optimized/current/ranger_core_db_oracle.sql > 021c34c77 > > security-admin/db/oracle/patches/055-add-syncSource-col-in-x_user-x_portal_user-x_group.sql > PRE-CREATION > security-admin/db/postgres/optimized/current/ranger_core_db_postgres.sql > 89e9e77a0 > > security-admin/db/postgres/patches/055-add-syncSource-col-in-x_user-x_portal_user-x_group.sql > PRE-CREATION > > security-admin/db/sqlanywhere/optimized/current/ranger_core_db_sqlanywhere.sql > b90a48e0a > > security-admin/db/sqlanywhere/patches/055-add-syncSource-col-in-x_user-x_portal_user-x_group.sql > PRE-CREATION > security-admin/db/sqlserver/optimized/current/ranger_core_db_sqlserver.sql > 7e03289dd > > security-admin/db/sqlserver/patches/055-add-syncSource-col-in-x_user-x_portal_user-x_group.sql > PRE-CREATION > security-admin/src/main/java/org/apache/ranger/biz/UserMgr.java e72a7a733 > security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java 38b06d1b2 > security-admin/src/main/java/org/apache/ranger/entity/XXGroup.java > 996a9e1d4 > security-admin/src/main/java/org/apache/ranger/entity/XXPortalUser.java > ed69f385e > security-admin/src/main/java/org/apache/ranger/entity/XXUser.java 0464e7b6e > security-admin/src/main/java/org/apache/ranger/rest/XUserREST.java > 4c648aeb1 > security-admin/src/main/java/org/apache/ranger/service/XGroupService.java > 44eebcfb7 > > security-admin/src/main/java/org/apache/ranger/service/XGroupServiceBase.java > cde91dc52 > > security-admin/src/main/java/org/apache/ranger/service/XPortalUserServiceBase.java > 85e4b8412 > security-admin/src/main/java/org/apache/ranger/service/XUserService.java > 0728eea2a > > security-admin/src/main/java/org/apache/ranger/service/XUserServiceBase.java > 9cdc14ebf > security-admin/src/main/java/org/apache/ranger/view/VXGroup.java ed6470425 > security-admin/src/main/java/org/apache/ranger/view/VXPortalUser.java > b6035b236 > security-admin/src/main/java/org/apache/ranger/view/VXUser.java 96f6468f7 > > ugsync-util/src/main/java/org/apache/ranger/ugsyncutil/model/XGroupInfo.java > a80e5dc1b > ugsync-util/src/main/java/org/apache/ranger/ugsyncutil/model/XUserInfo.java > bc1a6655c > > ugsync/src/main/java/org/apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java > 65e26fe06 > > > Diff: https://reviews.apache.org/r/73354/diff/4/ > > > Testing > ------- > > Tested the changes on postgres: > 1. New column is present on the tables x_user, x_portal_user and x_group. > 2. REST API calls (POST) made for user and group result in persistence of > data for syncSource: verified via db queries. > 3. The syncSource field appears in the response for a GET at the user and > group APIs with correct data. > > > Thanks, > > Abhishek Kumar > >
