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 2019/01/23 10:24:13 UTC

[GitHub] OmerJog edited a comment on issue #4476: [AIRFLOW-3552] Add ImapToS3TransferOperator

OmerJog edited a comment on issue #4476: [AIRFLOW-3552] Add ImapToS3TransferOperator
URL: https://github.com/apache/airflow/pull/4476#issuecomment-456748527
 
 
   @ashb Actually most with Operator.
   I'm not sure why the Transfer operators are treated differently at the end they are still Operators like the rest:
   `PostgresToGoogleCloudStorageOperator`
   `SFTPToS3Operator`
   etc..
   
   Actually there is a 3rd kind   `HiveToDynamoDBTransferOperator `in file  `hive_to_dynamodb.py`
   
   I think there should be specific rules for this in `CONTRIBUTING.md` and that will be enforced when code-reviewing. But this discussion is out of scope of this specific PR. 
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services