Skip to content

Commit

Permalink
fix: remove sqllineage_lib from redshift and tableau
Browse files Browse the repository at this point in the history
  • Loading branch information
sagar-salvi-apptware committed Dec 4, 2024
1 parent 02e7b10 commit 8f9db11
Showing 1 changed file with 1 addition and 11 deletions.
12 changes: 1 addition & 11 deletions metadata-ingestion/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -150,13 +150,6 @@
| classification_lib
)

sqllineage_lib = {
"sqllineage==1.3.8",
# We don't have a direct dependency on sqlparse but it is a dependency of sqllineage.
# There have previously been issues from not pinning sqlparse, so it's best to pin it.
# Related: https://github.com/reata/sqllineage/issues/361 and https://github.com/reata/sqllineage/pull/360
"sqlparse==0.4.4",
}

aws_common = {
# AWS Python SDK
Expand Down Expand Up @@ -207,7 +200,6 @@
"sqlalchemy-redshift>=0.8.3",
"GeoAlchemy2",
"redshift-connector>=2.1.0",
*sqllineage_lib,
*path_spec_common,
}

Expand Down Expand Up @@ -492,9 +484,7 @@
"slack": slack,
"superset": superset_common,
"preset": superset_common,
# FIXME: I don't think tableau uses sqllineage anymore so we should be able
# to remove that dependency.
"tableau": {"tableauserverclient>=0.24.0"} | sqllineage_lib | sqlglot_lib,
"tableau": {"tableauserverclient>=0.24.0"} | sqlglot_lib,
"teradata": sql_common
| usage_common
| sqlglot_lib
Expand Down

0 comments on commit 8f9db11

Please sign in to comment.