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

Ability to not log every request, optimizations, db_index, and a management command #31

Merged
merged 10 commits into from
Aug 17, 2014

Conversation

JoshData
Copy link
Contributor

Hope this is helpful.

  • adds a setting to not log every request, which makes this easier to use on a high-load site
  • use Python's cProfile class to profile all method calls in a request (with a setting to turn it on)
  • adds a management command to clear the request/response/etc. tables

and

  • doesn't compute a reverse or load the configuration on every request
  • adds some indexes to the models
  • tweaks templates, README
  • on the SQL queries page, display the query start time as relative to the request start time

@mtford90
Copy link
Collaborator

This looks awesome Josh, cheers - limiting num. requests intercepted is a great shout and really like the idea of integrating cProfile.

I'll merge this all in tommorow and will fix the broken build - think it's the usual python 2/3 stuff (iteritems etc).

@JoshData
Copy link
Contributor Author

Fantastic.

Thanks for getting this project going!

mtford90 added a commit that referenced this pull request Aug 17, 2014
Ability to not log every request, optimizations, db_index, and a management command
@mtford90 mtford90 merged commit 746f70f into jazzband:master Aug 17, 2014
@mtford90
Copy link
Collaborator

My pleasure =).

Just merged this in and released to PyPi as 0.4. I took the liberty of making a few style changes to the cProfile stuff - just added scrolling for overflow and made Silk's own profiling section more explicit. The build fail was simply down to StringIO being moved into the io module in py >3 so no issues there

Thanks again!

pablodiazgutierrez pushed a commit to appfluence/django-silk that referenced this pull request Oct 18, 2020
* Add regression tests for jazzband#22

* Handle more complicated PATCH calls
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants