[PATCH] LDAP: When resolving a SID, search for groups first, then users

[Jakub Hrozek]

This is a performance enhancement for the SID2name resolution. On most
clients, resolving a group SID to name is much more frequent operation
as it's performed during login to resolve a group SID. Therefore we
should try the group first and then users.

The attached patch switches the order of operations. No other functional
change is present there.

[Ondrej Kos]

On 07/16/2013 08:24 PM, Jakub Hrozek wrote:
>  This is a performance enhancement for the SID2name resolution. On
> most
>  clients, resolving a group SID to name is much more frequent operation
>  as it's performed during login to resolve a group SID. Therefore we
>  should try the group first and then users.
>
>  The attached patch switches the order of operations. No other functional
>  change is present there.
>
>
>
>  _______________________________________________
>  sssd-devel mailing list
>  sssd-devel(a)lists.fedorahosted.org
>  https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
>  
The patch looks and applies good, Ack

Ondra

-- 
Ondrej Kos
Associate Software Engineer
Identity Management - SSSD
Red Hat Czech

[Jakub Hrozek]

On Wed, Jul 17, 2013 at 10:22:18AM +0200, Ondrej Kos wrote:
>  On 07/16/2013 08:24 PM, Jakub Hrozek wrote:
>  >This is a performance enhancement for the SID2name resolution. On most
>  >clients, resolving a group SID to name is much more frequent operation
>  >as it's performed during login to resolve a group SID. Therefore we
>  >should try the group first and then users.
>  >
>  >The attached patch switches the order of operations. No other functional
>  >change is present there.
>  >
>  >
>  >
>  >_______________________________________________
>  >sssd-devel mailing list
>  >sssd-devel(a)lists.fedorahosted.org
>  >https://lists.fedorahosted.org/mailman/listinfo/sssd-devel
>  >
>  
>  The patch looks and applies good, Ack
>  
>  Ondra 
Pushed to master.