Merge lp:~kalebral-deactivatedaccount/drizzle-automation/fix-drizzle-sql-query into lp:drizzle-automation

Proposed by Lee Bieber
Status: Merged
Approved by: Patrick Crews
Approved revision: 218
Merge reported by: Patrick Crews
Merged at revision: not available
Proposed branch: lp:~kalebral-deactivatedaccount/drizzle-automation/fix-drizzle-sql-query
Merge into: lp:drizzle-automation
Diff against target: 11 lines (+1/-1)
1 file modified
drizzle/automation/lib/db.py (+1/-1)
To merge this branch: bzr merge lp:~kalebral-deactivatedaccount/drizzle-automation/fix-drizzle-sql-query
Reviewer Review Type Date Requested Status
Patrick Crews Approve
Review via email: mp+41640@code.launchpad.net

Description of the change

fix problem when using drizzle and a sql query fails, was referrencing output variable which doesn't exist for drizzle"

To post a comment you must log in.
Revision history for this message
Patrick Crews (patrick-crews) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'drizzle/automation/lib/db.py'
--- drizzle/automation/lib/db.py 2010-11-15 21:54:06 +0000
+++ drizzle/automation/lib/db.py 2010-11-23 18:37:38 +0000
@@ -103,7 +103,7 @@
103 try:103 try:
104 db.query(sql)104 db.query(sql)
105 except:105 except:
106 logging.error("SQL Execution error executing SQL:\n%s\n%s" % (sql, output))106 logging.error("SQL Execution error executing SQL:\n%s" % (sql))
107 db.close()107 db.close()
108 sys.exit(1)108 sys.exit(1)
109109

Subscribers

People subscribed via source and target branches

to all changes: