Opened 12 years ago

Last modified 4 years ago

#9363 new New feature

Add a traceback to each query in CursorDebugWrapper

Reported by: Gergely Kontra Owned by: nobody
Component: Database layer (models, ORM) Version: 1.0
Severity: Normal Keywords: debug, query
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no


I'm not a python wizard, so I'm just suspecting the following is possible:

When you examine connection.queries, it would be great to write out the file and the line, from the Query comes from.
Ok, one must define where the query created (the first time you say filter on a manager, the last time ...), but this should help greatly figuring out why certain queries are executed.

Change History (10)

comment:1 Changed 11 years ago by Jacob

Triage Stage: UnreviewedAccepted

comment:2 Changed 9 years ago by Luke Plant

Severity: Normal
Type: New feature

comment:3 Changed 8 years ago by Aymeric Augustin

UI/UX: unset

Change UI/UX from NULL to False.

comment:4 Changed 8 years ago by Aymeric Augustin

Easy pickings: unset

Change Easy pickings from NULL to False.

comment:5 Changed 6 years ago by Claude Paroz

#22320 was a duplicate with a patch. I think the idea is good, however I'm a bit worried about the memory consumption, even if production sites should not be affected. IMO, fixing #12581 is a prerequisite to this one.

comment:7 Changed 6 years ago by Claude Paroz

Has patch: set

comment:8 Changed 6 years ago by Aymeric Augustin

Resolution: wontfix
Status: newclosed

This feature is provided by the Django Debug Toolbar. It even works for templates. If you need this feature, just install the Debug Toolbar.

I know what the code looks like, and I'd rather leave it up to third-parties...

comment:9 Changed 6 years ago by Malcolm Box

Django debug toolbar doesn't help for any situation where you're not running via a web browser, or in a non-request cycle. So for example, any test case that uses assertNumQueries (see #22320 for example)

The patch for this is a simple few lines, so I don't understand your concern about "what the code looks like"

comment:10 Changed 4 years ago by Tim Graham

Patch needs improvement: set
Resolution: wontfix
Status: closednew
Summary: [feature request] track down where a query is requested for debuggingAdd a traceback to each query in CursorDebugWrapper

Aymeric withdraws his wontfix.

PR with a few comments for improvement.

Note: See TracTickets for help on using tickets.
Back to Top