diff options
author | Federico Caselli <cfederico87@gmail.com> | 2021-04-21 22:49:09 +0200 |
---|---|---|
committer | Mike Bayer <mike_mp@zzzcomputing.com> | 2021-04-28 20:03:27 -0400 |
commit | d3c73ad8012e15bf47529b3fcb0bac1298fbdb90 (patch) | |
tree | 31ae2c420b9d58d55e56a9191a34176c1deb7c16 /lib/sqlalchemy/dialects/postgresql/base.py | |
parent | 1443945e61f1f113e46a5044315a91558d4d232a (diff) | |
download | sqlalchemy-d3c73ad8012e15bf47529b3fcb0bac1298fbdb90.tar.gz |
Propertly ignore ``Identity`` in MySQL and MariaDb.
Ensure that the MySQL and MariaDB dialect ignore the
:class:`_sql.Identity` construct while rendering the
``AUTO_INCREMENT`` keyword in a create table.
The Oracle and PostgreSQL compiler was updated to not render
:class:`_sql.Identity` if the database version does not support it
(Oracle < 12 and PostgreSQL < 10). Previously it was rendered regardless
of the database version.
Fixes: #6338
Change-Id: I2ca0902fdd7b4be4fc1a563cf5585504cbea9360
Diffstat (limited to 'lib/sqlalchemy/dialects/postgresql/base.py')
-rw-r--r-- | lib/sqlalchemy/dialects/postgresql/base.py | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/lib/sqlalchemy/dialects/postgresql/base.py b/lib/sqlalchemy/dialects/postgresql/base.py index 47a933479..26b025b1a 100644 --- a/lib/sqlalchemy/dialects/postgresql/base.py +++ b/lib/sqlalchemy/dialects/postgresql/base.py @@ -2464,6 +2464,11 @@ class PGDDLCompiler(compiler.DDLCompiler): if isinstance(impl_type, sqltypes.TypeDecorator): impl_type = impl_type.impl + has_identity = ( + column.identity is not None + and self.dialect.supports_identity_columns + ) + if ( column.primary_key and column is column.table._autoincrement_column @@ -2471,7 +2476,7 @@ class PGDDLCompiler(compiler.DDLCompiler): self.dialect.supports_smallserial or not isinstance(impl_type, sqltypes.SmallInteger) ) - and column.identity is None + and not has_identity and ( column.default is None or ( @@ -2498,12 +2503,12 @@ class PGDDLCompiler(compiler.DDLCompiler): if column.computed is not None: colspec += " " + self.process(column.computed) - if column.identity is not None: + if has_identity: colspec += " " + self.process(column.identity) - if not column.nullable and not column.identity: + if not column.nullable and not has_identity: colspec += " NOT NULL" - elif column.nullable and column.identity: + elif column.nullable and has_identity: colspec += " NULL" return colspec @@ -3086,6 +3091,8 @@ class PGDialect(default.DefaultDialect): supports_empty_insert = False supports_multivalues_insert = True + supports_identity_columns = True + default_paramstyle = "pyformat" ischema_names = ischema_names colspecs = colspecs @@ -3193,6 +3200,7 @@ class PGDialect(default.DefaultDialect): 9, 2, ) + self.supports_identity_columns = self.server_version_info >= (10,) def on_connect(self): if self.isolation_level is not None: |