Skip to content
This repository was archived by the owner on Mar 1, 2018. It is now read-only.

"Add Computer widget", Catalog caching and other small fixes#71

Merged
jnraine merged 6 commits into
jnraine:masterfrom
rickychilcott:master
Oct 24, 2011
Merged

"Add Computer widget", Catalog caching and other small fixes#71
jnraine merged 6 commits into
jnraine:masterfrom
rickychilcott:master

Conversation

@rickychilcott

Copy link
Copy Markdown
Contributor

Completed work on first draft of computer widget. The only thing to be done would be adding jQuery support for selecting computer group when adding the compter.

Added Catalog caching which should significantly speed up computer checkins.

Lastly, made a minor fix to boostrap:user to hide your password in the console and also added a TextMate Backtracer gem to make it nicer when linking to directly to TextMate for error backtraces.

Ricky Chilcott added 5 commits October 22, 2011 14:06
My experience has been that this will increase catalog request times,
at minimum, 3x to 5x faster.  On an extremely trivial catalog (i.e. one
package) the time spent in ActiveRecord queries went from 2.8 ms to .6
ms.  The overall request went from 364ms to 106ms.
@rickychilcott

Copy link
Copy Markdown
Contributor Author

Worked on Caching today, and this got 'auto-updated' into this pull request.

@rickychilcott

Copy link
Copy Markdown
Contributor Author

To add a bit more compelling information to catalog caching: 3 runs of generating the same catalog

1st: Completed 200 OK in 1347ms (Views: 0.8ms | ActiveRecord: 329.5ms)
2nd: Completed 200 OK in 139ms (Views: 0.6ms | ActiveRecord: 0.5ms)
3rd: Completed 200 OK in 72ms (Views: 0.6ms | ActiveRecord: 0.3ms)

This is with a non-trivial catalog of a little over 100 packages.

@jnraine jnraine merged commit c074336 into jnraine:master Oct 24, 2011
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants