Drizzle Automation

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

  1. fix-drizzle-sql-query
  2. Merge into trunk
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
1=== modified file 'drizzle/automation/lib/db.py'
2--- drizzle/automation/lib/db.py 2010-11-15 21:54:06 +0000
3+++ drizzle/automation/lib/db.py 2010-11-23 18:37:38 +0000
4@@ -103,7 +103,7 @@
5 try:
6 db.query(sql)
7 except:
8- logging.error("SQL Execution error executing SQL:\n%s\n%s" % (sql, output))
9+ logging.error("SQL Execution error executing SQL:\n%s" % (sql))
10 db.close()
11 sys.exit(1)
12

Subscribers

People subscribed via source and target branches

to all changes: