collective / pas.plugins.ldap

Zope (and Plone) PAS Plugin providing users and groups from LDAP directory
http://pypi.python.org/pypi/pas.plugins.ldap
Other
13 stars 20 forks source link

UGM caching #13

Closed thet closed 8 years ago

thet commented 9 years ago

caching rework:

thet commented 9 years ago

if we want memcached with plone.memoize: https://github.com/plone/Products.PloneOrg/blob/master/src/Products/PloneOrg/caching.py

jensens commented 9 years ago

ad cache ugm().users and ugm().groups: ok, but also only a workaround, we need to solve it, so that not all keys are loaded.

ad remove bda.cache: using plone.memoize sounds good at first. but if it comes to many users you dont want to mix ldap with template cache and all the other memoize cache in one memcached. so make sure its possible to have an own memcached configured for ldap.

rnixx commented 9 years ago

+1 for keeping the ability for a dedicated cache.

@thet ad settings - i need to take a look

thet commented 9 years ago

i see, makes sense. i'd love to see bda.cache having some default cache decorators, which use themselves a default caching backend adapter, adapting self from the decorated method. that would be as easy to use as plone.memoize but give all the flexibility you need.

but for the current deployment, i'll just use the thet-all branch which includes this fixe and live with these tape-fixed solutions until another project allows refactoring.

jensens commented 9 years ago

whats the state here?

thet commented 9 years ago

@jensens on hold. this branch works for a client project.

thet commented 8 years ago

obsolete. performance improvements are already in master.