]> git.ipfire.org Git - thirdparty/sqlalchemy/sqlalchemy.git/commit
explicitly fetch inserted pk for values(pkcol=None)
authorMike Bayer <mike_mp@zzzcomputing.com>
Mon, 9 May 2022 02:49:33 +0000 (22:49 -0400)
committerMike Bayer <mike_mp@zzzcomputing.com>
Mon, 9 May 2022 13:49:50 +0000 (09:49 -0400)
commitd972b0f4ed4cd55c3f8e422816b32e9081168513
treebed6eabf2708bf19a8d1024dd73b95672431653c
parent47c91d06b56b0a0cf366d3c1f8b6d71a82149e43
explicitly fetch inserted pk for values(pkcol=None)

Altered the compilation mechanics of the :class:`.Insert` construct such
that the "autoincrement primary key" column value will be fetched via
``cursor.lastrowid`` or RETURNING even if present in the parameter set or
within the :meth:`.Insert.values` method as a plain bound value, for
single-row INSERT statements on specific backends that are known to
generate autoincrementing values even when explicit NULL is passed. This
restores a behavior that was in the 1.3 series for both the use case of
separate parameter set as well as :meth:`.Insert.values`. In 1.4, the
parameter set behavior unintentionally changed to no longer do this, but
the :meth:`.Insert.values` method would still fetch autoincrement values up
until 1.4.21 where :ticket:`6770` changed the behavior yet again again
unintentionally as this use case was never covered.

The behavior is now defined as "working" to suit the case where databases
such as SQLite, MySQL and MariaDB will ignore an explicit NULL primary key
value and nonetheless invoke an autoincrement generator.

Fixes: #7998
Change-Id: I5d4105a14217945f87fbe9a6f2a3c87f6ef20529
doc/build/changelog/unreleased_20/7998.rst [new file with mode: 0644]
lib/sqlalchemy/dialects/mysql/base.py
lib/sqlalchemy/dialects/sqlite/base.py
lib/sqlalchemy/engine/default.py
lib/sqlalchemy/sql/compiler.py
lib/sqlalchemy/sql/crud.py
test/requirements.py
test/sql/test_insert.py
test/sql/test_insert_exec.py