Skip to content

Commit

Permalink
fix: Fix hanging downstream tests in tap-postgres (#1977)
Browse files Browse the repository at this point in the history
Investigate use of future=True
  • Loading branch information
edgarrmondragon committed Sep 22, 2023
1 parent bb98db5 commit fa25c46
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 2 deletions.
3 changes: 3 additions & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -267,6 +267,9 @@ select = [
]
src = ["samples", "singer_sdk", "tests"]
target-version = "py37"
unfixable = [
"ERA", # Don't remove commented out code
]

[tool.ruff.per-file-ignores]
"docs/conf.py" = [
Expand Down
2 changes: 0 additions & 2 deletions singer_sdk/connectors/sql.py
Original file line number Diff line number Diff line change
Expand Up @@ -325,7 +325,6 @@ def create_engine(self) -> Engine:
return sqlalchemy.create_engine(
self.sqlalchemy_url,
echo=False,
future=True,
json_serializer=self.serialize_json,
json_deserializer=self.deserialize_json,
)
Expand All @@ -336,7 +335,6 @@ def create_engine(self) -> Engine:
return sqlalchemy.create_engine(
self.sqlalchemy_url,
echo=False,
future=True,
)

def quote(self, name: str) -> str:
Expand Down

0 comments on commit fa25c46

Please sign in to comment.