Re: [Freeipa-devel] [PATCH] Add LDAPMultiQuery base class and make it the base of LDAPDelete

2010-10-01 Thread Adam Young
On 10/01/2010 10:02 AM, Rob Crittenden wrote: Pavel Zuna wrote: In other words: make *-del commands accept 1 or more primary keys of entries to be deleted. We can now delete more entries at a time with a single command. Ticket #20 The code looks ok but I have to nack the patch because I'm ass

Re: [Freeipa-devel] [PATCH] Add LDAPMultiQuery base class and make it the base of LDAPDelete

2010-10-01 Thread Rob Crittenden
Pavel Zuna wrote: In other words: make *-del commands accept 1 or more primary keys of entries to be deleted. We can now delete more entries at a time with a single command. Ticket #20 The code looks ok but I have to nack the patch because I'm assuming this is going to break existing unit tes

Re: [Freeipa-devel] [PATCH] Add LDAPMultiQuery base class and make it the base of LDAPDelete

2010-10-01 Thread Adam Young
On 10/01/2010 08:49 AM, Pavel Zuna wrote: In other words: make *-del commands accept 1 or more primary keys of entries to be deleted. We can now delete more entries at a time with a single command. Ticket #20 Pavel ACK. Pushed to master ___ Freei

[Freeipa-devel] [PATCH] Add LDAPMultiQuery base class and make it the base of LDAPDelete

2010-10-01 Thread Pavel Zuna
In other words: make *-del commands accept 1 or more primary keys of entries to be deleted. We can now delete more entries at a time with a single command. Ticket #20 Pavel >From 1aabeb75114ef01ec23044031664f82ed0364825 Mon Sep 17 00:00:00 2001 From: Pavel Zuna Date: Fri, 1 Oct 2010 12:35:27 -