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/08 12:14:21 UTC

[GitHub] [airflow] potiuk edited a comment on pull request #12916: Adds predefined providers to install_requires.

potiuk edited a comment on pull request #12916:
URL: https://github.com/apache/airflow/pull/12916#issuecomment-740583712


   > ![Screenshot from 2020-12-08 13-08-19](https://user-images.githubusercontent.com/595491/101482206-a4d00900-3956-11eb-914b-01c497a5bb78.png)
   > 
   > @RosterIn - I hope that will make it clear :)
   
   And the wording in "naming convention" is I think rather clear:
   
   > It is often debatable where to put transfer operators but we agreed to the following criteria:
   > * We use "maintainability" of the operators as the main criteria - so the transfer operator should be kept at the provider which has highest "interest" in the transfer operator
   > * For Cloud Providers or Service providers that usually means that the transfer operators should land at the "target" side of the transfer


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