URL: https://github.com/SSSD/sssd/pull/390
Title: #390: NSS: Add option to disable memcache

lslebodn commented:
"""
On (22/09/17 06:48), mzidek-rh wrote:
>@lslebodn I am not sure about that. Maybe we could document the fact that it 
>has significant performance impact? Retrospectively, I think we should do this 
>also for SSS_NSS_USE_MEMCACHE. It may be obvious to some people, but stressing 
>it out is better IMO.
>

SSS_NSS_USE_MEMCACHE is a different case. Because you intentionally disable it
per client. Disabling memcache globally on heavy load server (e.g. mailserver)
is like jumping from skyscraper without a parachute.

(I know what I am talking about https://pagure.io/SSSD/sssd/issue/3520)

Such misfeature should not have a place in upstream.
We should not repeat the same mistake as with allowing authentication without
tls. Ideal would be have this PR just as a downstream patch to make
one particular customer (with unreasonable request due to workarounds on top
of workarounds) happy.

LS

"""

See the full comment at 
https://github.com/SSSD/sssd/pull/390#issuecomment-331454131
_______________________________________________
sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org
To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

Reply via email to