patches for time limited getncchanges

Matthieu Patou mat at
Tue Nov 22 12:38:17 MST 2011

On 21/11/2011 20:30, Stefan (metze) Metzmacher wrote:
> Hi Matthieu,
> I've pushed my latest version of it here:
>> I squashed what was needed and added the needed code for
>> replmd_replicated_apply_merge, I pushed all the code in my repo in the
>> drs branch:
>> I think it should be ok, it does the job both (in term of adding the
>> isRecycled) for new objects and updated objects.
> I think we need to test how replication works if an object is restored.
> Does it just work by removing the 'isDeleted' attribute or by setting it
> to 'FALSE'?
As I tested and as this page 
indicate it's globaly, you replace the DN and you remove the isDeleted 
> In case of replication I guess the meta data stamp for 'isDeleted' will
> always be
> there, so checking for the existence of the related stamp, may not indicate
> the object has isDeleted: TRUE.
Right, that's why It was the correct move to test if the recycle-bin is 
enabled because if it's the case then we shouldn't do anything while 
replicating or in the kcc.

> In replmd_replicated_apply_merge() we also need to cover the case, where
> the isDeleted: TRUE is already in the local ldb.
> I think we need to calculate how the new object will look like first
> and then check if we still need to add isRecycled: TRUE.
I don't think it's needed, if we are replicating for the first time  a 
deleted object, we just check it hasn't the isRecycled attribute and 
also hasn't the lastknownRDN and the recyclebin is not activated.
For a replication bringing the tombstone (isDeleted=TRUE) state to an 
object that we already know we have to check that the same rules and set 
the isRecycled, for all the other cases we do not need to do something.

> metze

Matthieu Patou
Samba Team

More information about the samba-technical mailing list