ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sailaja Polavarapu <spolavar...@hortonworks.com>
Subject Re: Review Request 66294: RANGER-1985: Code changes to support follow-up tasks
Date Tue, 03 Apr 2018 01:01:46 GMT

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

(Updated April 3, 2018, 1:01 a.m.)


Review request for ranger, Ankita Sinha, Gautam Borad, Abhay Kulkarni, Madhan Neethiraj, Pradeep
Agrawal, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, and Velmurugan Periasamy.


Changes
-------

Fixed PMD violations


Bugs: RANGER-1985
    https://issues.apache.org/jira/browse/RANGER-1985


Repository: ranger


Description
-------

– Add other meta-data to sync source popup (like group search first enabled)
– Distinguish between full sync and incremental sync events
– Show synctime, last modified time in event time format
– In addition to #users/groups sync'ed from source, show added/updated users (explore this
to see how best to show this without confusing users)


Diffs (updated)
-----

  security-admin/db/mysql/optimized/current/ranger_core_db_mysql.sql 65fc51cc 
  security-admin/db/mysql/patches/031-create-schema-for-usersync-audit-info.sql 2405a3ef 
  security-admin/db/oracle/optimized/current/ranger_core_db_oracle.sql 0634776f 
  security-admin/db/oracle/patches/031-create-schema-for-usersync-audit-info.sql e03e8937

  security-admin/db/postgres/optimized/current/ranger_core_db_postgres.sql 70f4bcc7 
  security-admin/db/postgres/patches/031-create-schema-for-usersync-audit-info.sql 2227f011

  security-admin/db/sqlanywhere/optimized/current/ranger_core_db_sqlanywhere.sql d59788cc

  security-admin/db/sqlanywhere/patches/031-create-schema-for-usersync-audit-info.sql 26d857b5

  security-admin/db/sqlserver/optimized/current/ranger_core_db_sqlserver.sql f67f1094 
  security-admin/db/sqlserver/patches/031-create-schema-for-usersync-audit-info.sql 362f4152

  security-admin/src/main/java/org/apache/ranger/entity/XXUgsyncAuditInfo.java 56098338 
  security-admin/src/main/java/org/apache/ranger/service/XUgsyncAuditInfoService.java 85a96512

  security-admin/src/main/java/org/apache/ranger/service/XUgsyncAuditInfoServiceBase.java
bf6f6e15 
  security-admin/src/main/java/org/apache/ranger/view/VXFileSyncSourceInfo.java d8c66ca1 
  security-admin/src/main/java/org/apache/ranger/view/VXLdapSyncSourceInfo.java fd2d8e11 
  security-admin/src/main/java/org/apache/ranger/view/VXUgsyncAuditInfo.java 6fef928a 
  security-admin/src/main/java/org/apache/ranger/view/VXUnixSyncSourceInfo.java 47bd839d 
  ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapDeltaUserGroupBuilder.java
7d85f33b 
  ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapPolicyMgrUserGroupBuilder.java
54a519a9 
  ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapUserGroupBuilder.java 85cba3c5

  ugsync/src/main/java/org/apache/ranger/unixusersync/config/UserGroupSyncConfig.java 45eeb1bc

  ugsync/src/main/java/org/apache/ranger/unixusersync/model/FileSyncSourceInfo.java 02387e11

  ugsync/src/main/java/org/apache/ranger/unixusersync/model/LdapSyncSourceInfo.java 318138d4

  ugsync/src/main/java/org/apache/ranger/unixusersync/model/UgsyncAuditInfo.java 9dfd3a55

  ugsync/src/main/java/org/apache/ranger/unixusersync/model/UnixSyncSourceInfo.java 97b95a1f

  ugsync/src/main/java/org/apache/ranger/unixusersync/process/FileSourceUserGroupBuilder.java
a50b0c76 
  ugsync/src/main/java/org/apache/ranger/unixusersync/process/PolicyMgrUserGroupBuilder.java
9d3112dd 
  ugsync/src/main/java/org/apache/ranger/unixusersync/process/UnixUserGroupBuilder.java c58589ef

  ugsync/src/main/java/org/apache/ranger/usergroupsync/UserGroupSync.java f078cb8d 
  ugsync/src/test/java/org/apache/ranger/usergroupsync/PolicyMgrUserGroupBuilderTest.java
312ea9b8 


Diff: https://reviews.apache.org/r/66294/diff/3/

Changes: https://reviews.apache.org/r/66294/diff/2-3/


Testing
-------

1. Performed functional tests with unix, file, and LDAP/AD as sync sources
2. Performed functional tests with different combinations of group search and user search
with AD/LDAP as sync source
3. Also ran some functional tests with and without incremental sync and nested groups


Thanks,

Sailaja Polavarapu


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message