Code changes as per review comment

datahub-project · Sep 11, 2023 · 82617ed · 82617ed
1 parent 24fcc1f
commit 82617ed
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 12 deletions.
diff --git a/.github/workflows/dagster-plugin.yml b/.github/workflows/dagster-plugin.yml
@@ -31,17 +31,10 @@ jobs:
       DATAHUB_TELEMETRY_ENABLED: false
     strategy:
       matrix:
+        python-version: ["3.7", "3.10"]
         include:
           - python-version: "3.7"
-            extraPythonRequirement: "dagster>=1.3.3"
-          - python-version: "3.7"
-            extraPythonRequirement: "dagster>=1.3.3"
-          - python-version: "3.10"
-            extraPythonRequirement: "dagster>=1.3.3"
-          - python-version: "3.10"
-            extraPythonRequirement: "dagster>=1.3.3"
           - python-version: "3.10"
-            extraPythonRequirement: "dagster>=1.3.3"
       fail-fast: false
     steps:
       - uses: actions/checkout@v3

diff --git a/metadata-ingestion/setup.py b/metadata-ingestion/setup.py
@@ -392,9 +392,6 @@ def get_long_description():
     "powerbi-report-server": powerbi_report_server,
     "vertica": sql_common | {"vertica-sqlalchemy-dialect[vertica-python]==0.0.8"},
     "unity-catalog": databricks | sqllineage_lib,
-    "dagster": {
-        f"acryl-datahub-dagster-plugin == {package_metadata['__version__']}",
-    },
 }
 
 # This is mainly used to exclude plugins from the Docker image.
@@ -650,7 +647,6 @@ def get_long_description():
         "file = datahub.ingestion.reporting.file_reporter:FileReporter",
     ],
     "apache_airflow_provider": ["provider_info=datahub_provider:get_provider_info"],
-    "dagster_provider": ["provider_info=datahub_dagster_plugin:get_provider_info"],
 }