[SSSD] Behaviour of getgrnam/getgrgid

Stephen Gallagher sgallagh at redhat.com
Mon Sep 27 10:41:53 UTC 2010


-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 09/25/2010 09:12 AM, Simo Sorce wrote:
> On Thu, 23 Sep 2010 14:02:20 -0400
> Stephen Gallagher <sgallagh at redhat.com> wrote:
> 
>> I'm not a huge fan of firing off all of the sdap_process_group_state()
>> calls in a loop for (effectively) simultaneous operation. I'd prefer
>> if you implemented this with our _step() approach (where you handle
>> each one serially until they're all finished). It's quite a bit
>> easier to follow in a debugger. This will also eliminate the need for
>> GROUPMEMBER_REQ_PARALLEL. Believe it or not, it won't be any slower
>> because the LDAP id op operations will serialize the requests anyway
>> so we don't open large numbers of separate connections to the LDAP
>> server.
> 
> Steve, I agree with Ralf's approach here.
> Yes it makes the code a bit more complex, but it will kill latency big
> time. And latency is something that hurts us really bad.


After a weekend's thought, I agree as well. The value is worth the cost.

- -- 
Stephen Gallagher
RHCE 804006346421761

Delivering value year after year.
Red Hat ranks #1 in value among software vendors.
http://www.redhat.com/promo/vendor/
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.10 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAkygdPEACgkQeiVVYja6o6OcOACfclUHY1BBsG6nJvcB+OuNI+lL
s3gAn16f0scOx0woTBZQkxImk+RuktZy
=aS/t
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list