[GitHub] [airflow] potiuk commented on pull request #20245: Exclude snowflake-sqlalchemy v1.2.5

2022-01-12 Thread GitBox
potiuk commented on pull request #20245: URL: https://github.com/apache/airflow/pull/20245#issuecomment-1011849274 https://github.com/apache/airflow/blob/constraints-2.2.3/constraints-3.7.txt#L485 -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [airflow] potiuk commented on pull request #20245: Exclude snowflake-sqlalchemy v1.2.5

2022-01-12 Thread GitBox
potiuk commented on pull request #20245: URL: https://github.com/apache/airflow/pull/20245#issuecomment-1011848901 @NadimYounes -Indeed 2.2.3 constraints were prepared before snowflake was yanked. It should be fixed now. -- This is an automated message from the Apache Git Service. To

[GitHub] [airflow] potiuk commented on pull request #20245: Exclude snowflake-sqlalchemy v1.2.5

2021-12-13 Thread GitBox
potiuk commented on pull request #20245: URL: https://github.com/apache/airflow/pull/20245#issuecomment-992389303 We should actually even release an ad-hoc release of the snowflake Provider with those limitiations. I will take a look at that after we release the databricks one. -- This

[GitHub] [airflow] potiuk commented on pull request #20245: Exclude snowflake-sqlalchemy v1.2.5

2021-12-13 Thread GitBox
potiuk commented on pull request #20245: URL: https://github.com/apache/airflow/pull/20245#issuecomment-992387108 I updated the updated constraints (snowflake-sqlalchemy = 1.2.4) for Airlfow 2.2.2 and 2.1.4. I think this is the right way to go - we should encourage people to go to the

[GitHub] [airflow] potiuk commented on pull request #20245: Exclude snowflake-sqlalchemy v1.2.5

2021-12-13 Thread GitBox
potiuk commented on pull request #20245: URL: https://github.com/apache/airflow/pull/20245#issuecomment-992351737 Constraints updated in main. Merged. -- 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

[GitHub] [airflow] potiuk commented on pull request #20245: Exclude snowflake-sqlalchemy v1.2.5

2021-12-13 Thread GitBox
potiuk commented on pull request #20245: URL: https://github.com/apache/airflow/pull/20245#issuecomment-992316059 Generating the constraints now -> I will be back in ~ 1.5 h and I will merge/push constraints then (getting 3rd vaccination in 30 mins :) ) -- This is an automated message

[GitHub] [airflow] potiuk commented on pull request #20245: Exclude snowflake-sqlalchemy v1.2.5

2021-12-13 Thread GitBox
potiuk commented on pull request #20245: URL: https://github.com/apache/airflow/pull/20245#issuecomment-992310839 Also @mik-laj -> I believe we should fix constraints for released airlfow version - snowflake sqlalchemy 1.2.5 is used in all constraints - 2.1.3, 2.1.4, 2.2.0, 2.2.1, 2.2.2.

[GitHub] [airflow] potiuk commented on pull request #20245: Exclude snowflake-sqlalchemy v1.2.5

2021-12-13 Thread GitBox
potiuk commented on pull request #20245: URL: https://github.com/apache/airflow/pull/20245#issuecomment-992307146 I will also cherry-pick that one to 2.2.3 @jedcunningham -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [airflow] potiuk commented on pull request #20245: Exclude snowflake-sqlalchemy v1.2.5

2021-12-13 Thread GitBox
potiuk commented on pull request #20245: URL: https://github.com/apache/airflow/pull/20245#issuecomment-992305671 This one will need to get constraints regenerated quickly after merge as this will generate conflicts with main constraints. Let me do it now and I can merge/push constraints