[GitHub] feluelle commented on issue #4476: [AIRFLOW-3552] Add ImapToS3TransferOperator

[GitBox <gi...@apache.org>]

feluelle commented 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` and then just naming the attachments like the list you could pass in to `s3_key` 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