You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "killua1zoldyck (via GitHub)" <gi...@apache.org> on 2023/06/12 09:30:50 UTC

[GitHub] [airflow] killua1zoldyck commented on a diff in pull request #31837: Add realtime container execution logs for BatchOperator

killua1zoldyck commented on code in PR #31837:
URL: https://github.com/apache/airflow/pull/31837#discussion_r1226366284


##########
airflow/providers/amazon/aws/hooks/batch_waiters.py:
##########
@@ -194,6 +201,9 @@ def wait_for_job(self, job_id: str, delay: int | float | None = None) -> None:
 
         :param delay:  A delay before polling for job status
 
+        :param get_batch_log_fetcher: A method that returns batch_log_fetcher of
+            type AwsTaskLogFetcher or None.

Review Comment:
   I have updated it. Thanks!



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