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 2020/12/28 00:01:50 UTC

[GitHub] [airflow] r-richmond opened a new pull request #13333: Move snowflake limitations to snowflake

r-richmond opened a new pull request #13333:
URL: https://github.com/apache/airflow/pull/13333


   This PR follows on #13286 by moving the other package limitations that snowflake brings to the snowflake option.
   
   This limits the impacts that snowflake's peculiarities has on the airflow user base to only those who use snowflake.
   
   I have attempted to revert all of the package limitations to as they were before they went in with #12636
   * Cryptography: https://github.com/apache/airflow/commit/7ef3e7a296ead9dd499a6524ecfbfa0d67f9131c
   * Requests: https://github.com/apache/airflow/pull/12636/files#diff-fa602a8a75dc9dcc92261bac5f533c2a85e34fcceaff63b3a3a81d9acde2fc52L122
   
   @potiuk fyi
   
   Note: Bumped the epoch number as specified in the [comment](https://github.com/apache/airflow/blob/v1-10-stable/setup.py#L428)


----------------------------------------------------------------
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.

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



[GitHub] [airflow] potiuk edited a comment on pull request #13333: Move snowflake limitations to snowflake

Posted by GitBox <gi...@apache.org>.
potiuk edited a comment on pull request #13333:
URL: https://github.com/apache/airflow/pull/13333#issuecomment-751532840


   This does not work, unfortunately. For one, we also have `requests` limitations in setup.cfg . If we do not add it there `pip install --upgrade --upgrade-strategy eager` will produce conflicting requirements.


----------------------------------------------------------------
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.

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



[GitHub] [airflow] potiuk edited a comment on pull request #13333: Move snowflake limitations to snowflake

Posted by GitBox <gi...@apache.org>.
potiuk edited a comment on pull request #13333:
URL: https://github.com/apache/airflow/pull/13333#issuecomment-751532840


   This does not work, unfortunately. For one, we also have `requests` limitations in install_requires . If we do not add it there `pip install --upgrade --upgrade-strategy eager` will produce conflicting requirements.


----------------------------------------------------------------
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.

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



[GitHub] [airflow] potiuk commented on pull request #13333: Move snowflake limitations to snowflake

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


   This does not work, unfortunately. For one, we also have limitations in setup.cfg . If we do not add it there `pip install --upgrade --upgrade-strategy eager` will produce conflicting requirements.


----------------------------------------------------------------
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.

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



[GitHub] [airflow] potiuk commented on pull request #13333: Move snowflake limitations to snowflake

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


   I think i have found a way to achieve it and continue the --eager upgrade working . Stary tuned @r-richmond 


----------------------------------------------------------------
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.

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



[GitHub] [airflow] r-richmond closed pull request #13333: Move snowflake limitations to snowflake

Posted by GitBox <gi...@apache.org>.
r-richmond closed pull request #13333:
URL: https://github.com/apache/airflow/pull/13333


   


----------------------------------------------------------------
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.

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