LP_DEBUG_SQL broken

Bug #834466 reported by Данило Шеган
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Gary Poster

Bug Description

With Gary's changes for ++profile++ to include data akin to LP_DEBUG_SQL(_EXTRA), neither of these variables output any SQL statements anymore:

$ LP_DEBUG_SQL=1 make harness
bin/harness
execute_zcml_for_scripts()...
Reading $PYTHONSTARTUP...
Initializing storm...
Creating the factory...

>>> from lp.bugs.model.bug import Bug
>>> Bug.get(1)
<Bug at 0x807f350>
>>>

Reverting r13771 made them re-appear for me again:

$ bzr merge -r13771..13770 .
$ LP_DEBUG_SQL=1 make harness
bin/harness
execute_zcml_for_scripts()...
Reading $PYTHONSTARTUP...
Initializing storm...
Creating the factory...

>>> from lp.bugs.model.bug import Bug
>>> Bug.get(1)
SELECT Bug.date_last_message, Bug.date_last_updated, Bug.date_made_private, Bug.datecreated, Bug.description, Bug.duplicateof, Bug.heat, Bug.heat_last_updated, Bug.id, Bug.latest_patch_uploaded, Bug.message_count, Bug.name, Bug.number_of_duplicates, Bug.owner, Bug.private, Bug.security_related, Bug.title, Bug.users_affected_count, Bug.users_unaffected_count, Bug.who_made_private FROM Bug WHERE Bug.id = 1 LIMIT 1
----------------------------------------------------------------------
<Bug at 0x711f090>
>>>

Now, this is a regression, but not a user facing regression, so I am not sure if it should be marked Critical.

Tags: qa-ok

Related branches

Revision history for this message
Данило Шеган (danilo) wrote :

Gary, assigning to you just so you can share your opinion.

Changed in launchpad:
assignee: nobody → Gary Poster (gary)
Changed in launchpad:
assignee: Gary Poster (gary) → Данило Шеган (danilo)
status: Triaged → In Progress
Changed in launchpad:
assignee: Данило Шеган (danilo) → Gary Poster (gary)
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :
tags: added: qa-needstesting
Changed in launchpad:
status: In Progress → Fix Committed
Gary Poster (gary)
tags: added: qa-ok
removed: qa-needstesting
William Grant (wgrant)
Changed in launchpad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.