You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/08/05 22:59:06 UTC

[GitHub] [airflow] jedcunningham opened a new pull request #17452: Pin snowflake-sqlalchemy

jedcunningham opened a new pull request #17452:
URL: https://github.com/apache/airflow/pull/17452


   snowflake-sqlalchemy 1.3 now depends on sqlalchemy 1.4+, so pin to a max of 1.2.x.
   
   The webserver cannot start when we have sqlalchemy 1.4, throwing the following exceptions:
   
   ```airflow._vendor.connexion.exceptions.ResolverError: <ResolverError: columns>```
   
   and
   
   ```AttributeError: columns```
   
   A workaround for any users trying to install `apache-airflow-providers-snowflake`, simply also add `snowflake-sqlalchemy==1.2.5` (as of writing) to avoid pulling in the latest version of `snowflake-sqlalchemy`, and as a result, the latest `sqlalchemy`.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] jedcunningham closed pull request #17452: Pin snowflake-sqlalchemy

Posted by GitBox <gi...@apache.org>.
jedcunningham closed pull request #17452:
URL: https://github.com/apache/airflow/pull/17452


   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] jedcunningham commented on pull request #17452: Pin snowflake-sqlalchemy

Posted by GitBox <gi...@apache.org>.
jedcunningham commented on pull request #17452:
URL: https://github.com/apache/airflow/pull/17452#issuecomment-894270773


   Good call @potiuk, done.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] jedcunningham commented on pull request #17452: Pin snowflake-sqlalchemy

Posted by GitBox <gi...@apache.org>.
jedcunningham commented on pull request #17452:
URL: https://github.com/apache/airflow/pull/17452#issuecomment-894542307


   This ended up being an issue with the legacy pip resolver, so I don't think we actually need this after all.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] github-actions[bot] commented on pull request #17452: Pin snowflake-sqlalchemy

Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #17452:
URL: https://github.com/apache/airflow/pull/17452#issuecomment-894422732


   The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest main at your convenience, or amend the last commit of the PR, and push it with --force-with-lease.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org