[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (comment)

2016-09-14 Thread jhrozek
jhrozek commented on a pull request """ * master: 3319d964721396c07daba383ded6aaaf33ed6e3b """ See the full comment at https://github.com/SSSD/sssd/pull/22#issuecomment-246954357 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (comment)

2016-09-14 Thread jhrozek
jhrozek commented on a pull request """ On Tue, Sep 13, 2016 at 01:33:49AM -0700, mzidek-rh wrote: > Makes sense, but I would prefer to add new "else if" branch with > ADMINLIMIT_EXCEEDED specific debug message. > > It would also make sense to change the SDAP_SRCH_FLG_SIZELIMIT_SILENT into >

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (comment)

2016-09-13 Thread lslebodn
lslebodn commented on a pull request """ On (13/09/16 01:33), mzidek-rh wrote: >Makes sense, but I would prefer to add new "else if" branch with >ADMINLIMIT_EXCEEDED specific debug message. > >It would also make sense to change the SDAP_SRCH_FLG_SIZELIMIT_SILENT >into something more generic, like

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (comment)

2016-09-13 Thread mzidek-rh
mzidek-rh commented on a pull request """ Makes sense, but I would prefer to add new "else if" branch with ADMINLIMIT_EXCEEDED specific debug message. It would also make sense to change the SDAP_SRCH_FLG_SIZELIMIT_SILENT into something more generic, like SDAP_SRCH_FLG_SILENT because we want to