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

Review request for ranger, bhavik patel, Dhaval Shah, Kishor Gollapalliwar, 
Abhay Kulkarni, Mehul Parikh, Pradeep Agrawal, Sailaja Polavarapu, and 
Velmurugan Periasamy.


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


Repository: ranger


Description
-------

There is a change in external user 'status' (i.e x_portal_user tables column) 
which are getting synced into ranger admin, default ‘status’ value of synced 
users are getting set as 0(disabled) which was not the case in ranger-2.1. This 
is the behaviour change between 2.1 and later versions, Probably due this 
(https://github.com/apache/ranger/commit/9be6a7d01f8d6a83e740ac27225eda1fff83501a).

- Added change to mark external users status as enable(1).
- written a java patch to update the status of existing external users.


Diffs
-----

  security-admin/db/mysql/optimized/current/ranger_core_db_mysql.sql b852f43dd 
  security-admin/db/oracle/optimized/current/ranger_core_db_oracle.sql 
3704ff598 
  security-admin/db/postgres/optimized/current/ranger_core_db_postgres.sql 
be009d239 
  
security-admin/db/sqlanywhere/optimized/current/ranger_core_db_sqlanywhere.sql 
2f4db3e0e 
  security-admin/db/sqlserver/optimized/current/ranger_core_db_sqlserver.sql 
f566955c6 
  security-admin/src/main/java/org/apache/ranger/biz/UserMgr.java dbddef598 
  security-admin/src/main/java/org/apache/ranger/db/XXPortalUserDao.java 
b4fdf027b 
  
security-admin/src/main/java/org/apache/ranger/patch/PatchForExternalUserStatusUpdate_J10056.java
 PRE-CREATION 
  security-admin/src/main/resources/META-INF/jpa_named_queries.xml b2a748f2c 


Diff: https://reviews.apache.org/r/73961/diff/1/


Testing
-------

- All unit tests passed
- Verified fresh and upgrade(existing external users status are getting updated 
to 1) scenario.


Thanks,

Mateen Mansoori

Reply via email to