diff options
| author | Mike Bayer <mike_mp@zzzcomputing.com> | 2008-10-07 16:58:53 +0000 |
|---|---|---|
| committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2008-10-07 16:58:53 +0000 |
| commit | ff3df488f5d84a1fa6288adaaa48948fe1614e7d (patch) | |
| tree | e6adc6e3c2bcbcd8abef18666e1e72a246660ede | |
| parent | 86c39923183a5b7a5a63860cadfdf524b2edf01a (diff) | |
| download | sqlalchemy-ff3df488f5d84a1fa6288adaaa48948fe1614e7d.tar.gz | |
- Oracle will detect string-based statements which contain
comments at the front before a SELECT as SELECT statements.
[ticket:1187]
| -rw-r--r-- | CHANGES | 5 | ||||
| -rw-r--r-- | lib/sqlalchemy/databases/oracle.py | 5 | ||||
| -rw-r--r-- | test/dialect/oracle.py | 4 |
3 files changed, 13 insertions, 1 deletions
@@ -74,7 +74,10 @@ CHANGES - mysql - Temporary tables are now reflectable. - +- oracle + - Oracle will detect string-based statements which contain + comments at the front before a SELECT as SELECT statements. + [ticket:1187] 0.5.0rc1 ======== diff --git a/lib/sqlalchemy/databases/oracle.py b/lib/sqlalchemy/databases/oracle.py index 3f5077d1a..eb02d4505 100644 --- a/lib/sqlalchemy/databases/oracle.py +++ b/lib/sqlalchemy/databases/oracle.py @@ -14,6 +14,8 @@ from sqlalchemy.sql import operators as sql_operators, functions as sql_function from sqlalchemy import types as sqltypes +SELECT_REGEXP = re.compile(r'(\s*/\*\+.*?\*/)?\s*SELECT', re.I | re.UNICODE) + class OracleNumeric(sqltypes.Numeric): def get_col_spec(self): if self.precision is None: @@ -206,6 +208,9 @@ class OracleExecutionContext(default.DefaultExecutionContext): self.out_parameters[name] = self.cursor.var(dbtype) self.parameters[0][name] = self.out_parameters[name] + def returns_rows_text(self, statement): + return SELECT_REGEXP.match(statement) + def create_cursor(self): c = self._connection.connection.cursor() if self.dialect.arraysize: diff --git a/test/dialect/oracle.py b/test/dialect/oracle.py index ac52b26a9..6803508f5 100644 --- a/test/dialect/oracle.py +++ b/test/dialect/oracle.py @@ -356,6 +356,10 @@ class SequenceTest(TestBase, AssertsCompiledSQL): seq = Sequence("My_Seq", schema="Some_Schema") assert dialect.identifier_preparer.format_sequence(seq) == '"Some_Schema"."My_Seq"' +class ExecuteTest(TestBase): + __only_on__ = 'oracle' + def test_basic(self): + assert testing.db.execute("/*+ this is a comment */ SELECT 1 FROM DUAL").fetchall() == [(1,)] if __name__ == '__main__': testenv.main() |
