daloradius icon indicating copy to clipboard operation
daloradius copied to clipboard
verified publisher icon lirantal

fix by Felix Kurth <[email protected]> to address slow page loads with >70k users

Open lirantal opened this issue 8 years ago • 2 comments

Quoting Felix:

Dear Mr Tal,

Our company uses its Daloradius software with about 70,000 users. Because a lot of very slow sql queries we have made some adjustments to increase the performance. We would be very happy if you would install this in future versions.

The changes apply to both the queries for counting and for the output of the data are responsible.

This PR addresses the issue by applying the patches he sent me.

lirantal avatar Mar 03 '18 18:03 lirantal

btw @MiguelVis, do you want to review the changes here? I couldn't test it locally, but if you have a setup ready then this PR seems like a promising performance fix.

lirantal avatar Feb 15 '19 18:02 lirantal

@lirantal

Ok, I will look at this asap.

MiguelVis avatar Feb 15 '19 20:02 MiguelVis

@filippolauria do you maybe want to try this PR out? It seems to be a really big performance improvement fix but it needs a bit of testing so maybe you can confirm this works ok and then land the changes?

lirantal avatar Oct 13 '22 14:10 lirantal

Agree. Feel free to merge the commit!

lirantal avatar Oct 17 '22 09:10 lirantal

This has been addressed in other PRs.

filippolauria avatar Dec 14 '22 15:12 filippolauria