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/12/09 17:17:47 UTC

[GitHub] [airflow] eladkal commented on pull request #20167: Organise S3 Classes in Amazon Provider

eladkal commented on pull request #20167:
URL: https://github.com/apache/airflow/pull/20167#issuecomment-990054748


   @KennyRich can you please rebase & resolve conflicts?
   Also note that some of the S3 operators are also in contrib.
   For example:
   
   https://github.com/apache/airflow/blob/f4e04c713f56b6cd95d9e88c53f577f785838a83/tests/deprecated_classes.py#L1067-L1068
   
   This means that contrib (which is deprecated) is now pointing to deprecated class. This is probably wrong and should direct to the new class directly(?)


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