ranger-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ayub Pathan <aryanayub...@gmail.com>
Subject Re: Review Request 58961: reducing the highest time stamp value to pick all the users syncd during sync cycle
Date Wed, 03 May 2017 13:19:26 GMT

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




ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapDeltaUserGroupBuilder.java
Line 550 (original), 550 (patched)
<https://reviews.apache.org/r/58961/#comment246770>

    With the current fix, search record frequency is set to 1sec. Do you think, this might
impact the server as the delta sync will happen for every 1 second now, as oppossed to 1 min?
    
    We need to understand the frequency where the users are added within a minute of last
sync cycle. Also, the impact if the newly added user is available after 1 min delay.


- Ayub Pathan


On May 3, 2017, 12:50 p.m., deepak sharma wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/58961/
> -----------------------------------------------------------
> 
> (Updated May 3, 2017, 12:50 p.m.)
> 
> 
> Review request for ranger and Sailaja Polavarapu.
> 
> 
> Bugs: RANGER-1475
>     https://issues.apache.org/jira/browse/RANGER-1475
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> currently we increase the deltaSyncUserTimeStamp by 1 minute, in order to avoid search
record repetition for next sync cycle, but if there is a new user added within a minute of
last sync cycle then it does not sync, so reducing the increament time from 1min to 1sec to
avoid this issue.
> 
> 
> Diffs
> -----
> 
>   ugsync/src/main/java/org/apache/ranger/ldapusersync/process/LdapDeltaUserGroupBuilder.java
d65ba52 
> 
> 
> Diff: https://reviews.apache.org/r/58961/diff/1/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> deepak sharma
> 
>


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