You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by po...@apache.org on 2022/08/30 10:56:16 UTC

[airflow] branch main updated: Replace SQL with Common SQL in pre commit (#26058)

This is an automated email from the ASF dual-hosted git repository.

potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new 8acdc2a834 Replace SQL with Common SQL in pre commit (#26058)
8acdc2a834 is described below

commit 8acdc2a834b9c4e287fe612ed56ab8908d777609
Author: eladkal <45...@users.noreply.github.com>
AuthorDate: Tue Aug 30 13:56:03 2022 +0300

    Replace SQL with Common SQL in pre commit (#26058)
    
    This pre-commit verify source/target provider for transfer exist.
    Now that we have `common.sql` provider there is no need to use SQL along with Common SQL
---
 airflow/providers/amazon/provider.yaml                        | 2 +-
 airflow/providers/google/provider.yaml                        | 4 ++--
 airflow/providers/slack/provider.yaml                         | 2 +-
 scripts/ci/pre_commit/pre_commit_check_provider_yaml_files.py | 2 +-
 4 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/airflow/providers/amazon/provider.yaml b/airflow/providers/amazon/provider.yaml
index 53db674a3a..df7ccd556c 100644
--- a/airflow/providers/amazon/provider.yaml
+++ b/airflow/providers/amazon/provider.yaml
@@ -533,7 +533,7 @@ transfers:
     target-integration-name: Amazon Simple Storage Service (S3)
     how-to-guide: /docs/apache-airflow-providers-amazon/operators/transfer/local_to_s3.rst
     python-module: airflow.providers.amazon.aws.transfers.local_to_s3
-  - source-integration-name: SQL
+  - source-integration-name: Common SQL
     target-integration-name: Amazon Simple Storage Service (S3)
     how-to-guide: /docs/apache-airflow-providers-amazon/operators/transfer/sql_to_s3.rst
     python-module: airflow.providers.amazon.aws.transfers.sql_to_s3
diff --git a/airflow/providers/google/provider.yaml b/airflow/providers/google/provider.yaml
index e72bee14cd..5923805302 100644
--- a/airflow/providers/google/provider.yaml
+++ b/airflow/providers/google/provider.yaml
@@ -817,7 +817,7 @@ transfers:
     target-integration-name: Google Cloud Storage (GCS)
     how-to-guide: /docs/apache-airflow-providers-google/operators/transfer/trino_to_gcs.rst
     python-module: airflow.providers.google.cloud.transfers.trino_to_gcs
-  - source-integration-name: SQL
+  - source-integration-name: Common SQL
     target-integration-name: Google Cloud Storage (GCS)
     python-module: airflow.providers.google.cloud.transfers.sql_to_gcs
   - source-integration-name: Google Cloud Storage (GCS)
@@ -896,7 +896,7 @@ transfers:
     target-integration-name: Google Spreadsheet
     how-to-guide: /docs/apache-airflow-providers-google/operators/transfer/gcs_to_sheets.rst
     python-module: airflow.providers.google.suite.transfers.gcs_to_sheets
-  - source-integration-name: SQL
+  - source-integration-name: Common SQL
     target-integration-name: Google Spreadsheet
     how-to-guide: /docs/apache-airflow-providers-google/operators/transfer/sql_to_sheets.rst
     python-module: airflow.providers.google.suite.transfers.sql_to_sheets
diff --git a/airflow/providers/slack/provider.yaml b/airflow/providers/slack/provider.yaml
index 57843af639..08809ac8a8 100644
--- a/airflow/providers/slack/provider.yaml
+++ b/airflow/providers/slack/provider.yaml
@@ -62,7 +62,7 @@ hooks:
       - airflow.providers.slack.hooks.slack_webhook
 
 transfers:
-  - source-integration-name: SQL
+  - source-integration-name: Common SQL
     target-integration-name: Slack
     python-module: airflow.providers.slack.transfers.sql_to_slack
     how-to-guide: /docs/apache-airflow-providers-slack/operators/sql_to_slack.rst
diff --git a/scripts/ci/pre_commit/pre_commit_check_provider_yaml_files.py b/scripts/ci/pre_commit/pre_commit_check_provider_yaml_files.py
index 756682be5e..eb87fa5618 100755
--- a/scripts/ci/pre_commit/pre_commit_check_provider_yaml_files.py
+++ b/scripts/ci/pre_commit/pre_commit_check_provider_yaml_files.py
@@ -83,7 +83,7 @@ def get_all_integration_names(yaml_files) -> List[str]:
     all_integrations = [
         i['integration-name'] for f in yaml_files.values() if 'integrations' in f for i in f["integrations"]
     ]
-    all_integrations += ["SQL", "Local"]
+    all_integrations += ["Local"]
     return all_integrations