ashb commented on a change in pull request #4524: [AIRFLOW-3707] Group subpackages by cloud providers URL: https://github.com/apache/airflow/pull/4524#discussion_r248058460
########## File path: airflow/utils/log/s3_task_handler.py ########## @@ -45,7 +45,7 @@ def _build_hook(self): except Exception: self.log.error( 'Could not create an S3Hook with connection id "%s". ' - 'Please make sure that airflow[s3] is installed and ' Review comment: Everything is an API - so the suffix is redundant to my mind. ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services