whummer opened a new pull request, #44079:
URL: https://github.com/apache/airflow/pull/44079

   First of all - thanks so much for building Airflow, it is an absolutely 
awesome platform! 🙌 🚀 
   
   This PR extends the functionality of `SnowflakeHook` to make the host/port 
configurable for Snowflake connections. This facilitates testing against 
non-standard Snowflake endpoints, for example when testing against 
[LocalStack](https://www.localstack.cloud/localstack-for-snowflake) (a local 
cloud emulator).
   
   A similar PR has been created against the Astronomer Cosmos repo some time 
ago (see https://github.com/astronomer/astronomer-cosmos/pull/1063), and our 
users have now requested to use the standard Airflow Snowflake connector with 
LocalStack Snowflake as well.
   
   The PR also updates the documentation for the Snowflake connector, but it 
does not contain any tests at this point. If adding a test is required, then 
any pointers or guidance would be appreciated.
   
   ---
   **^ Add meaningful description above**
   Read the **[Pull Request 
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-guidelines)**
 for more information.
   In case of fundamental code changes, an Airflow Improvement Proposal 
([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvement+Proposals))
 is needed.
   In case of a new dependency, check compliance with the [ASF 3rd Party 
License Policy](https://www.apache.org/legal/resolved.html#category-x).
   In case of backwards incompatible changes please leave a note in a 
newsfragment file, named `{pr_number}.significant.rst` or 
`{issue_number}.significant.rst`, in 
[newsfragments](https://github.com/apache/airflow/tree/main/newsfragments).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to