eladkal commented on PR #23979:
URL: https://github.com/apache/airflow/pull/23979#issuecomment-1146260469

   Case 3 + 4 of [AIP 
21](https://cwiki.apache.org/confluence/display/AIRFLOW/AIP-21%3A+Changes+in+import+paths):
   
   > In case of transfer operators where two providers are involved, the 
transfer operators will be moved to "source" of the transfer. When there is 
only one provider as target but source is a database or another non-provider 
source, the operator is put to the target provider.
   
   From that description I think the source is the right choice 
   
   On a larger scope discussion - The fact that we both need to look up where 
operator should be is a problem. If we are not sure I think users are even more 
unsure where to look. I think it's time that we will consider a simpler rules 
like "it's will **always** be in the target name". e.g XtoY will always be in Y 
provider (Y is guaranteed to be exist while X is not). I will raise this in the 
mailing list as an modification to AIP-21 (just as soon as mailing list will be 
a bit more quite as there many parallel discussions right now).
   
   Waiting for your confirmation before merging.


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