Re: [Freeipa-devel] [PATCH] 811 performance: faster DN implementation

2016-02-23 Thread Martin Basti
On 22.12.2015 16:59, Petr Spacek wrote: On 14.4.2015 19:32, Petr Vobornik wrote: Pushed to master: 11bd9d96f191066f7ba760549f00179c128a9787 Please be so kind and fix naming. AFAIK the patch refers to 'openldap' DN format but in fact it is python-ldap-ishm. It will surely confuse next

Re: [Freeipa-devel] [PATCH] 811 performance: faster DN implementation

2015-12-22 Thread Petr Spacek
On 14.4.2015 19:32, Petr Vobornik wrote: > > Pushed to master: 11bd9d96f191066f7ba760549f00179c128a9787 > >> >> Please be so kind and fix naming. AFAIK the patch refers to 'openldap' DN >> format but in fact it is python-ldap-ishm. It will surely confuse next >> generation of FreeIPA developers

Re: [Freeipa-devel] [PATCH] 811 performance: faster DN implementation

2015-04-14 Thread Petr Vobornik
On 04/14/2015 09:32 AM, Petr Spacek wrote: On 10.4.2015 13:11, Petr Viktorin wrote: ACK Pushed to master: 11bd9d96f191066f7ba760549f00179c128a9787 Please be so kind and fix naming. AFAIK the patch refers to 'openldap' DN format but in fact it is python-ldap-ishm. It will surely confuse

Re: [Freeipa-devel] [PATCH] 811 performance: faster DN implementation

2015-04-09 Thread Petr Vobornik
On 04/02/2015 11:54 AM, Petr Viktorin wrote: On 03/31/2015 12:11 PM, Petr Vobornik wrote: The only different thing is a lack of utf-8 encoded str support(as input). I don't know how much important the support is. I don't think that support is too important (assuming IPA doesn't use it!).

Re: [Freeipa-devel] [PATCH] 811 performance: faster DN implementation

2015-04-02 Thread Petr Viktorin
On 03/31/2015 12:11 PM, Petr Vobornik wrote: The only different thing is a lack of utf-8 encoded str support(as input). I don't know how much important the support is. I don't think that support is too important (assuming IPA doesn't use it!). However, the behavior with this patch is

Re: [Freeipa-devel] [PATCH] 811 performance: faster DN implementation

2015-04-01 Thread Petr Spacek
On 31.3.2015 12:11, Petr Vobornik wrote: The major change is that DN is no longer internally composed of RDNs and AVAs but it rather keeps the data in open ldap format - the same as output of str2dn function. Therefore, for immutable DNs, no other transformations are required on