Similar to #2851664: Make watchdog optional for performance, for very large sites, this can be a performance hindrance when loading a ton of users (in our case, 6500+).

I'm not sure how often settings need to be edited for directory rules, but caching this response should help with loading those forms. In addition, I added a checkbox to the preference pane that can make this step optional since you do have to clear cache if you're looking for specific users.

Attached are screenshots showing the load times with and without cache, and you can see an significant improvement with caching.

Without caching
Without caching

With caching
With Caching

CommentFileSizeAuthor
#2 cache-get-users-2851804-1.patch3.03 KBkyletaylored
Screen Shot 2017-02-11 at 5.20.15 PM.png96.34 KBkyletaylored
Screen Shot 2017-02-11 at 5.12.20 PM.png96.41 KBkyletaylored
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

kyletaylored created an issue. See original summary.

kyletaylored’s picture

kyletaylored CreditAttribution: kyletaylored at LevelTen Interactive commented
Status: Active » Needs review
FileSize
cache-get-users-2851804-1.patch3.03 KB

Patch attached.

kyletaylored’s picture

kyletaylored CreditAttribution: kyletaylored at LevelTen Interactive commented
Issue summary: View changes
kyletaylored’s picture

kyletaylored CreditAttribution: kyletaylored at LevelTen Interactive commented
zanonmark’s picture

zanonmark CreditAttribution: zanonmark commented
Assigned: Unassigned » zanonmark

zanonmark’s picture

zanonmark CreditAttribution: zanonmark commented
Status: Needs review » Fixed
zanonmark’s picture

zanonmark CreditAttribution: zanonmark commented
Status: Fixed » Closed (fixed)

Thanks for this patch, too: I just committed it to the current -dev branch.

Thanks,
Marco