URL:
https://github.com/SSSD/sssd/pull/390
Title: #390: NSS: Add option to disable memcache
mzidek-rh commented:
"""
As for not putting this to upstream and only to the requested downstream distro, I do not
like it. We may end up backporting it to future versions of that downstream distro as well
(after rebases), which is IMO unnecessary burden for downstream maintainers. I do not
think it is worth it just to avoid having this in upstream. In general, I would like to
avoid "downstream only" patches as much as possible.
As for not documenting the option the same way as we do for the option to disable tls, I
do not like this either, but if other developers agree with that, I am OK with it. The
difference between the option to disable tls and this one is that disabling tls was added
for testing purposes, while this one is added to support actual customer's use case
and as such it should be IMO documented. But indeed, the current version of the man page
is probably not stressing out enough how severe the performance impact of using this
option is, so we can add a warning there.
I did not do any changes to the man page for now, the options are:
- do not document the option in man pages (Lukas likes)
- stress out that using it can have significant negative performance impact (Michal
likes)
I would like to hear from other developers what they think the best approach is before I
do any changes.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/390#issuecomment-331477185