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.

Sign up for GitHub

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

Jump to bottom

[App] Performance bottleneck on latestMessageEachGroups (1363ms) #661

Open
danlg opened this issue Oct 18, 2016 · 1 comment
Open

[App] Performance bottleneck on latestMessageEachGroups (1363ms) #661

danlg opened this issue Oct 18, 2016 · 1 comment
Assignees
@danlg @amngupta
Labels
performance

Comments

@danlg
Copy link
Owner

danlg commented Oct 18, 2016 •
edited
Loading

See managing-waittime

https://kadira.io/academy/meteor-performance-101/content/managing-waittime
https://kadira.io/academy/meteor-performance-101/content/debugging-meteor-response-time-bottlenecks
https://kadira.io/academy/meteor-performance-101/content/know-your-observers

https://ui.kadira.io/apps/o8PeR6djxXFvDPyKc/dashboard/pubsub?selection=smartix%3Amessages%2FlatestMessageEachGroups&metric=resTime&type=pub&explore=314ff541-f3e1-446c-9779-a6dcaa420834

image

image
amngupta added a commit that referenced this issue Oct 20, 2016
@amngupta
[App] Performance bottleneck on latestMessageEachGroups (1363ms) #661
43747cb
@amngupta
Copy link
Collaborator

Before

screenshot from 2016-10-20 14 51 07

After

screenshot from 2016-10-20 16 14 08

amngupta added a commit that referenced this issue Oct 20, 2016
@amngupta
[App] Performance bottleneck on latestMessageEachGroups #661
9b7ffa5 
- Fixed latestMessageEachGroups publish function to return only the lastMessages to client
- Made other publications async
- Removed ClassPanel.onCreated function repatition
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@danlg danlg

@amngupta amngupta

Labels
performance
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@danlg @amngupta