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 2022/09/09 14:19:23 UTC

[GitHub] [airflow] nikhi-suthar commented on a diff in pull request #26206: Glue job continuously Cloudwatch logging in Airflow

nikhi-suthar commented on code in PR #26206:
URL: https://github.com/apache/airflow/pull/26206#discussion_r967125725


##########
airflow/providers/amazon/aws/hooks/glue.py:
##########
@@ -144,40 +146,39 @@ def get_job_state(self, job_name: str, run_id: str) -> str:
         job_run = glue_client.get_job_run(JobName=job_name, RunId=run_id, PredecessorsIncluded=True)
         return job_run['JobRun']['JobRunState']
 
+
     def print_job_logs(
         self,
         job_name: str,
         run_id: str,
+        log_client,

Review Comment:
   agree, I have made change and opening new Pull request since this branch get corrupt due to multiple changes and not able to merge with all changes. 



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