-
Notifications
You must be signed in to change notification settings - Fork 9
improve performance for group page #142
Comments
I've been wondering about that a bit too... In metamaps, we bootstrap the inclusion of the models into the page, but we On Thu, Apr 21, 2016 at 4:25 PM, Eugene Lynch notifications@github.com
Connor Turland |
I've made a pull request that will help with this: #146 There's likely couple more refactorings to make. |
MembershipsController#index and MembershipsController#my_memberships are also problems, I'll take a look at them too. I think it's the same problem with balance calculations. |
Just looking at the responses in the slow queries, they are all hitting the group serializer. So let's get #146 merged in and then look if there are further optimisations to make. |
@data-doge @gardner can we close this? |
42 seconds is too long for the group page to load
The text was updated successfully, but these errors were encountered: