URL: https://github.com/SSSD/sssd/pull/27 Title: #27: Minor code changes
lslebodn commented: """ I finally tried the patch and here are logs before change ``` [sssd[be[LDAP]]] [sdap_get_generic_op_finished] (0x0400): Search result: Administrative limit exceeded(11), no errmsg set [sssd[be[LDAP]]] [sdap_get_generic_op_finished] (0x0080): LDAP sizelimit was exceeded, returning incomplete data ``` and after change ``` [sssd[be[LDAP]]] [sdap_get_generic_op_finished] (0x0400): Search result: Administrative limit exceeded(11), no errmsg set [sssd[be[LDAP]]] [sdap_get_generic_op_finished] (0x0080): LDAP server limit was exceeded, returning incomplete data: Administrative limit exceeded(11) ``` Personally, I do not see any benefit. I would prefer to reject the patch. I will wait few days before closing this PR. """ See the full comment at https://github.com/SSSD/sssd/pull/27#issuecomment-262816103
_______________________________________________ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org