diff options
author | Federico Caselli <cfederico87@gmail.com> | 2022-04-11 23:19:16 +0200 |
---|---|---|
committer | Federico Caselli <cfederico87@gmail.com> | 2022-04-13 22:29:55 +0200 |
commit | c154ed5e047113ce6763cc5a26741f6a5022eb1d (patch) | |
tree | c295a93f303e920b7c647f48fbe53de77146a4db /lib/sqlalchemy/dialects/postgresql/_psycopg_common.py | |
parent | fd052732604a0d03167978215f0077e60b75851d (diff) | |
download | sqlalchemy-c154ed5e047113ce6763cc5a26741f6a5022eb1d.tar.gz |
Fix psycopg2 pre_ping with autocommit
Fixed an issue what would cause autocommit mode to be reset
when using pre_ping in conjunction engine level autocommit
on the psycopg2 driver.
Fixes: #7930
Change-Id: I4cccaf1b7f8cbacd853689458080784114fcc390
Diffstat (limited to 'lib/sqlalchemy/dialects/postgresql/_psycopg_common.py')
-rw-r--r-- | lib/sqlalchemy/dialects/postgresql/_psycopg_common.py | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/lib/sqlalchemy/dialects/postgresql/_psycopg_common.py b/lib/sqlalchemy/dialects/postgresql/_psycopg_common.py index 7f936fefb..e7d5e77c3 100644 --- a/lib/sqlalchemy/dialects/postgresql/_psycopg_common.py +++ b/lib/sqlalchemy/dialects/postgresql/_psycopg_common.py @@ -170,15 +170,17 @@ class _PGDialect_common_psycopg(PGDialect): def do_ping(self, dbapi_connection): cursor = None + before_autocommit = dbapi_connection.autocommit try: - self._do_autocommit(dbapi_connection, True) + if not before_autocommit: + self._do_autocommit(dbapi_connection, True) cursor = dbapi_connection.cursor() try: cursor.execute(self._dialect_specific_select_one) finally: cursor.close() - if not dbapi_connection.closed: - self._do_autocommit(dbapi_connection, False) + if not before_autocommit and not dbapi_connection.closed: + self._do_autocommit(dbapi_connection, before_autocommit) except self.dbapi.Error as err: if self.is_disconnect(err, dbapi_connection, cursor): return False |