Skip to content

Commit

Permalink
PostgresEngine is already a Generic, I don't think this adds anything
Browse files Browse the repository at this point in the history
Not completely convinced, but mypy is ok with it
  • Loading branch information
realtyem committed Oct 13, 2024
1 parent 22652eb commit d6b307c
Showing 1 changed file with 1 addition and 3 deletions.
4 changes: 1 addition & 3 deletions synapse/storage/engines/postgres.py
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@

import abc
import logging
from typing import TYPE_CHECKING, Any, Generic, Mapping, Optional, Tuple, cast
from typing import TYPE_CHECKING, Any, Mapping, Optional, Tuple, cast

from synapse.storage.engines._base import (
AUTO_INCREMENT_PRIMARY_KEYPLACEHOLDER,
Expand All @@ -36,12 +36,10 @@
if TYPE_CHECKING:
from synapse.storage.database import LoggingDatabaseConnection


logger = logging.getLogger(__name__)


class PostgresEngine(
Generic[ConnectionType, CursorType, IsolationLevelType],
BaseDatabaseEngine[ConnectionType, CursorType, IsolationLevelType],
metaclass=abc.ABCMeta,
):
Expand Down

0 comments on commit d6b307c

Please sign in to comment.