[SSSD] [PATCH] Remove duplicates detection in SRV resolving

Stephen Gallagher sgallagh at redhat.com
Mon Apr 11 20:09:32 UTC 2011


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

On 04/11/2011 12:07 PM, Stephen Gallagher wrote:
> On 04/11/2011 07:01 AM, Jakub Hrozek wrote:
>> The first patch fixes https://bugzilla.redhat.com/show_bug.cgi?id=694444
>> by not detecting duplicates at all.
> 
> Ack.
> 

Pushed the first patch to master and sssd-1-5.


>> The second patch is an optimization. Since it is possible that the list
>> of servers contains duplicates now, the code marks all duplicates when
>> setting a server status, too. I'm submitting it as a separate patch
>> because it might be easier to push only one of them if QE thinks they
>> have the capacity to test only one of them for the stable 1.5 branch.
> 
> Nack.
> 
> While it's not as if we call this function often, it seems
> counter-intuitive to me that we'd be looping through all the server
> lists (which could be arbitrarily long, if we're using SRV records) on
> every state update, rather than just eating this cost once when adding
> new servers to the list.
> 
> In other words, we should do a duplicate check before calling
> create_fo_server() and skip creating it if the name and port matches.
> 
_______________________________________________
sssd-devel mailing list
sssd-devel at lists.fedorahosted.org
https://fedorahosted.org/mailman/listinfo/sssd-devel

- -- 
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.11 (GNU/Linux)
Comment: Using GnuPG with Fedora - http://enigmail.mozdev.org/

iEYEARECAAYFAk2jX/sACgkQeiVVYja6o6OPLgCggqX0P092UPjIrdjlRCRVU6De
r58An1EM2u6u9x6JcPGckGj0IVqABhEn
=vzlg
-----END PGP SIGNATURE-----



More information about the sssd-devel mailing list