Opened 3 weeks ago

Last modified 2 weeks ago

#36882 assigned New feature

Add debug instrumentation for callproc() in CursorDebugWrapper

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

Description

The callproc() method in CursorDebugWrapper is not currently instrumented
for debug logging, unlike execute() and executemany(). This means stored
procedure calls are not logged when DEBUG=True.

This patch adds the callproc() method to CursorDebugWrapper to log
procedure calls with timing information, consistent with how other
database operations are logged.

The fix includes:

  • Implementation of callproc() in CursorDebugWrapper
  • Test to verify the instrumentation works correctly
  • Removal of the "XXX callproc isn't instrumented" comment

Change History (6)

comment:1 by Harsh007, 3 weeks ago

Owner: set to Harsh007
Status: new → assigned

comment:2 by Clifford Gama, 3 weeks ago

Keywords: stored procedures removed
Type: Bug → New feature
Version: 6.0 → dev

comment:3 by akshaytheflash, 3 weeks ago

Owner: changed from Harsh007 to akshaytheflash

comment:5 by Jacob Walls, 2 weeks ago

Needs documentation: set
Patch needs improvement: set
Triage Stage: Unreviewed → Accepted

Linters are failing, and needs a release note.

in reply to:  5 comment:6 by akshaytheflash, 2 weeks ago

I’m working on fixing the linters, adding a release note, and updating the documentation.
Please let me know if there are specific improvements you’d like in the patch.

Replying to Jacob Walls:

Linters are failing, and needs a release note.

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