From fdf5162e55a0dd22716fbd38c00ca0ac65e8811c Mon Sep 17 00:00:00 2001 From: Ville Brofeldt Date: Fri, 10 Dec 2021 14:37:27 +0200 Subject: [PATCH 1/2] fix(postgres): remove redundant tz factory --- superset/db_engine_specs/postgres.py | 7 ------- 1 file changed, 7 deletions(-) diff --git a/superset/db_engine_specs/postgres.py b/superset/db_engine_specs/postgres.py index b9217eb563a4e..44b4306c99a31 100644 --- a/superset/db_engine_specs/postgres.py +++ b/superset/db_engine_specs/postgres.py @@ -50,12 +50,6 @@ logger = logging.getLogger() -# Replace psycopg2.tz.FixedOffsetTimezone with pytz, which is serializable by PyArrow -# https://github.com/stub42/pytz/blob/b70911542755aeeea7b5a9e066df5e1c87e8f2c8/src/pytz/reference.py#L25 -class FixedOffsetTimezone(_FixedOffset): - pass - - # Regular expressions to catch custom errors CONNECTION_INVALID_USERNAME_REGEX = re.compile( 'role "(?P.*?)" does not exist' @@ -168,7 +162,6 @@ class PostgresBaseEngineSpec(BaseEngineSpec): def fetch_data( cls, cursor: Any, limit: Optional[int] = None ) -> List[Tuple[Any, ...]]: - cursor.tzinfo_factory = FixedOffsetTimezone if not cursor.description: return [] return super().fetch_data(cursor, limit) From 68f47f27ee9d5b9809d52915aa90f60647d2931f Mon Sep 17 00:00:00 2001 From: Ville Brofeldt Date: Fri, 10 Dec 2021 15:26:34 +0200 Subject: [PATCH 2/2] lint --- superset/db_engine_specs/postgres.py | 1 - 1 file changed, 1 deletion(-) diff --git a/superset/db_engine_specs/postgres.py b/superset/db_engine_specs/postgres.py index 44b4306c99a31..c2951c394d01c 100644 --- a/superset/db_engine_specs/postgres.py +++ b/superset/db_engine_specs/postgres.py @@ -32,7 +32,6 @@ ) from flask_babel import gettext as __ -from pytz import _FixedOffset from sqlalchemy.dialects.postgresql import ARRAY, DOUBLE_PRECISION, ENUM, JSON from sqlalchemy.dialects.postgresql.base import PGInspector from sqlalchemy.types import String, TypeEngine