[Freeipa-devel] [PATCH 0202-0203] Improve performance of initial LDAP synchronizationDetect end of initial LDAP synchronization phase

[Petr Spacek]

On 12.11.2013 16:13, Petr Spacek wrote:
> On 5.11.2013 12:29, Tomas Hozza wrote:
>> ----- Original Message -----
>>> Hello,
>>>
>>> Improve performance of initial LDAP synchronization.
>>>
>>> Changes are not journaled and SOA serial is not incremented during initial
>>> LDAP synchronization.
>>>
>>> This eliminates unnecessary synchronous writes to journal and also
>>> unnecessary SOA serial writes to LDAP.
>>>
>>> See commit messages and comments in syncrepl.c for all the gory details.
>>
>>
>> ACK.
>>
>> Patches look good. AXFR and IXFR works as expected. Also BIND starts up much
>> faster with these patches. Good job... :)
>>
>> Regards,
>>
>> Tomas
>
> Hmm, further testing revealed that patch 203 changed behavior little bit:
> Zones were loaded from LDAP correctly, but the SOA serial wasn't changed at
> all. As a result, zone transfers return inconsistent results if the data in
> LDAP are changed when BIND was not running.
>
> Patch 203-v2 imitates the old behavior from bind-dyndb-ldap 3.x: Zone serial
> is bumped *once* for each zone, so any changed in LDAP will be transferred
> correctly (with new serial).

Patch 202 v2 was rebased and fixes reconnection to LDAP and solves deadlock 
caused by too eager locking.

Patch 203 v3 was rebased and fixes reconnection to LDAP.

These patches should go to master branch.

-- 
Petr^2 Spacek

-------------- next part --------------
A non-text attachment was scrubbed...
Name: bind-dyndb-ldap-pspacek-0202-2-Detect-end-of-initial-LDAP-synchronization-phase.patch
Type: text/x-patch
Size: 20780 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20131213/fb9d263f/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: bind-dyndb-ldap-pspacek-0203-3-Improve-performance-of-initial-LDAP-synchronization.patch
Type: text/x-patch
Size: 7144 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20131213/fb9d263f/attachment-0001.bin>