decouple sql debug from debug and implement a verbose

Bug #1041308 reported by Therese McHale
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
Invalid
Wishlist
Therese McHale

Bug Description

Would be useful to decouple sql debug from normal debug. Also sqlalchemy supports the following debug levels using the echo parameter to create_engine:

1) echo=false (supposed to be identical to echo set to None, the default but doesn't behave well on version 0.6.4.3 of sqlalchemy
 so I've found the to be safer, see http://<email address hidden>/msg25071.html
for more detail. Glance is using echo=false , works fine on later sqlalchemy but problems on earlier version.

2) echo=true - the same as setting the the loglevel=debug i..e. logs sql statements

3) echo='debug' - returns result set as well as logging sql statements, currently not used by glance

Would be useful get 3) for glance using a sql_verbose option and I think because of 1) we should use the default rather than echo=false as this works better on older versions of sqlalchemy .

I will attach a patch which does the above

Revision history for this message
Therese McHale (therese-mchale) wrote :
Changed in glance:
assignee: nobody → Therese McHale (therese-mchale)
status: New → In Progress
Revision history for this message
Brian Waldon (bcwaldon) wrote :

Let's fix this in Grizzly

Changed in glance:
importance: Undecided → Wishlist
Revision history for this message
Therese McHale (therese-mchale) wrote :

Actually this can be closed now as it is possible on later versions using default_log_levels to decouple sqlalchemy debug from regular debug :

i.e. default_log_levels = sqlalchemy=DEBUG (or INFO/WARN)

effectively achieves the same result and thus it is no longer necessary to pursue this.

Alex Meade (alex-meade)
Changed in glance:
status: In Progress → Invalid
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.