ferruzzi commented on a change in pull request #17626:
URL: https://github.com/apache/airflow/pull/17626#discussion_r689714300
##########
File path: airflow/providers/amazon/aws/operators/ecs.py
##########
@@ -80,6 +84,70 @@ def list_tasks(self, cluster: str, launchType: str,
desiredStatus: str, family:
...
+class ECSTaskLogFetcher(Thread):
+ """
+ Fetches Cloudwatch log events with specific interval as a thread
+ and sends the log events to the info channel of the provided logger.
+ """
+
+ def __init__(
+ self,
+ *,
+ aws_conn_id: Optional[str] = None,
Review comment:
Nitpick request: This should now default to `'aws-default'` but it has
not been updated everywhere yet. **IFF** you are going to be making any
revisions, could you change this default? If not, I am going to be going
through these and standardizing that in the near future, so no biggie.
See:
http://airflow.apache.org/docs/apache-airflow-providers-amazon/stable/connections/aws.html
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
