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 2021/09/14 00:13:23 UTC

[GitHub] [airflow] o-nikolas commented on pull request #17145: Adds an s3 list prefixes operator

o-nikolas commented on pull request #17145:
URL: https://github.com/apache/airflow/pull/17145#issuecomment-918681350


   > @o-nikolas I went ahead and reverted back to having two distinct operators - if for no other reason than to keep this PR / convo alive. If I had a better understanding of s3 list keys / prefixes the use cases I would feel more confident exploring the first path you outlined above.
   > 
   > That being said, still open to workshopping this :)
   
   Sorry I've been on vacation for the past three weeks:
   
   Sounds good to me, I think that's perfectly valid :) I'll try have a review sometime this week.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org