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

Set up profiling and CI #136

Closed
stubailo opened this issue Apr 20, 2016 · 2 comments
Closed

Set up profiling and CI #136

stubailo opened this issue Apr 20, 2016 · 2 comments
Assignees

Comments

@stubailo
Copy link
Contributor

In order to have informed conversations about performance, we should do CPU profiling of stuff like inserting complex queries into the store, updating the store and rerunning queries, etc.

I'll start investigating some Node.js profiling tools that can help us measure this stuff.

Related to #128

@stubailo
Copy link
Contributor Author

To do this, we probably need to collect more real world scenarios, and it might not be really worth it until we tackle some of the more low-hanging fruit. If we think about making serious changes to the implementation, we should ask for supporting data first, or set up the profiling then.

@helfer
Copy link
Contributor

helfer commented Dec 5, 2016

🍂 we can reopen if necessary

@helfer helfer closed this as completed Dec 5, 2016
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants