Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

support prefetching of users and groups #17

Closed
wants to merge 2 commits into from

Conversation

chaoflow
Copy link

@chaoflow chaoflow commented Apr 14, 2016

For example for the sharing view in combination with LDAP it is very expensive to fetch user and group information with individual requests. To solve this we implemented for 4teamwork the possibility to prefetch users and groups for PAS plugins that support it.

relates:

@jensens
Copy link
Member

jensens commented Mar 7, 2018

Hence there was no activity for a long time, I propose to close this PR within next two weeks. If you do not feel ok with this, please speak up and provide us a rough schedule.

@jensens jensens reopened this Mar 7, 2018
@chaoflow
Copy link
Author

chaoflow commented Mar 8, 2018

@jensens The performance improvement of this and the related pull requests is significant and it would be a pity if they don't make it into plone. It seems it lacks somebody reviewing it. Not sure who should/could provide a schedule.

cc @jone

@djay
Copy link
Member

djay commented Mar 9, 2018

@chaoflow Any pros and cons for this? Some timings maybe? I think this change would really help us but it would be good to know if there are any knock on affects and any alternatives?

@chaoflow
Copy link
Author

@jone do you have input on the performance and any feedback from using the code?

@jensens jensens closed this May 21, 2019
@jensens jensens reopened this May 21, 2019
@jensens
Copy link
Member

jensens commented Dec 22, 2021

There was no activity on this for a while. I close this request for now. If you plan to work further on this please reopen and go on.

@jensens jensens closed this Dec 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants