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/10 19:08:53 UTC

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

feluelle edited a comment on issue #4476: [AIRFLOW-3552] Add ImapToS3TransferOperator
URL: https://github.com/apache/airflow/pull/4476#issuecomment-453213757
 
 
   PTAL: @feng-tao @kaxil @ashb 
   
   Especially because of this:
   > NOTE: This operator only transfers the latest attachment by name.
   
   I am thinking of changing this so that you can transfer multiple attachments that for example match the regular expression you pass in but I don't know if this is really a good idea and if so how would be the best way to implement this.
   
   Something like adding `imap_latest_only` (set to False) + passing `s3_bucket` so that the attachments just keep its names looks very strange to me. :/ ..but I don't know any other way of doing this.

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