sqlalchemy 0.6.1 breaks firebird driver

Issue #1823 resolved
Former user created an issue

Hi,

SQLalchemy 0.6.1 breaks executing direct statements with firebird:

>>> import sqlalchemy
>>> from sqlalchemy import create_engine
>>> sqlalchemy.__version__
'0.6.1'
>>> engine = create_engine('firebird://sysdba:masterkey@127.0.0.1:3050/testdb?type_conv=300',echo=True)
>>>
... conn = engine.connect()
>>>
... a = conn.execute("SELECT * FROM RDB$DATABASE")
2010-06-09 12:26:37,902 INFO sqlalchemy.engine.base.Engine.0x...75d0 SELECT * FROM RDB$DATABASE
2010-06-09 12:26:37,904 INFO sqlalchemy.engine.base.Engine.0x...75d0 ()
Traceback (most recent call last):
  File "<stdin>", line 2, in <module>
  File "/Users/chris/Development/sqltest/lib/python2.6/site-packages/SQLAlchemy-0.6.1-py2.6.egg/sqlalchemy/engine/base.py", line 1157, in execute
    params)
  File "/Users/chris/Development/sqltest/lib/python2.6/site-packages/SQLAlchemy-0.6.1-py2.6.egg/sqlalchemy/engine/base.py", line 1253, in _execute_text
    return self.__execute_context(context)
  File "/Users/chris/Development/sqltest/lib/python2.6/site-packages/SQLAlchemy-0.6.1-py2.6.egg/sqlalchemy/engine/base.py", line 1268, in __execute_context
    context.parameters[0](0), context=context)
  File "/Users/chris/Development/sqltest/lib/python2.6/site-packages/SQLAlchemy-0.6.1-py2.6.egg/sqlalchemy/engine/base.py", line 1360, in _cursor_execute
    context)
TypeError: do_execute() takes exactly 4 arguments (5 given)

This is due to a change in SQLAlchemy-0.6.1-py2.6.egg/sqlalchemy/engine/base.py The mysql engine has a different syntax

1351     def _cursor_execute(self, cursor, statement, parameters, context=None):

1355         try:
1356             self.dialect.do_execute(
1357                                 cursor,
1358                                 statement,
1359                                 parameters,
1360                                 context)

(old was:

self.dialect.do_execute(cursor, statement, parameters, context=context)

Thanks,

Christian

Comments (3)

  1. Mike Bayer repo owner

    yah seems like 0.6.2 will have to be a quick release, a couple others like this are in there too

  2. Log in to comment