Skip to content
This repository has been archived by the owner on May 17, 2024. It is now read-only.

Commit

Permalink
Merge pull request #232 from datafold/fix_pip_extras
Browse files Browse the repository at this point in the history
Fix for pip extras (e.g. pip install data-diff[snowflake])
  • Loading branch information
erezsh authored Sep 13, 2022
2 parents 77ce613 + 933af8c commit 15f897e
Show file tree
Hide file tree
Showing 3 changed files with 44 additions and 37 deletions.
2 changes: 1 addition & 1 deletion data_diff/databases/databricks.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
from .base import MD5_HEXDIGITS, CHECKSUM_HEXDIGITS, Database, import_helper, _query_conn, parse_table_name


@import_helper("databricks")
@import_helper(text="You can install it using 'pip install databricks-sql-connector'")
def import_databricks():
import databricks.sql

Expand Down
70 changes: 35 additions & 35 deletions poetry.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

9 changes: 8 additions & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,13 @@ dsnparse = "*"
click = "^8.1"
rich = "*"
toml = "^0.10.2"
mysql-connector-python = {version="*", optional=true}
psycopg2 = {version="*", optional=true}
snowflake-connector-python = {version="^2.7.2", optional=true}
cryptography = {version="*", optional=true}
trino = {version="^0.314.0", optional=true}
presto-python-client = {version="*", optional=true}
clickhouse-driver = {version="*", optional=true}

[tool.poetry.dev-dependencies]
parameterized = "*"
Expand All @@ -52,7 +59,7 @@ postgresql = ["psycopg2"]
snowflake = ["snowflake-connector-python", "cryptography"]
presto = ["presto-python-client"]
oracle = ["cx_Oracle"]
databricks = ["databricks-sql-connector"]
# databricks = ["databricks-sql-connector"]
trino = ["trino"]
clickhouse = ["clickhouse-driver"]
vertica = ["vertica-python"]
Expand Down

0 comments on commit 15f897e

Please sign in to comment.