airflow-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [airflow] sreenath-kamath commented on a change in pull request #17339: Adding support for multiple task-ids in the external task sensor
Date Mon, 02 Aug 2021 06:23:31 GMT

sreenath-kamath commented on a change in pull request #17339:
URL: https://github.com/apache/airflow/pull/17339#discussion_r680684052



##########
File path: airflow/sensors/external_task.py
##########
@@ -149,7 +167,10 @@ def poke(self, context, session=None):
         serialized_dttm_filter = ','.join(dt.isoformat() for dt in dttm_filter)
 
         self.log.info(
-            'Poking for %s.%s on %s ... ', self.external_dag_id, self.external_task_id, serialized_dttm_filter
+            'Poking for %s.%s on %s ... ',

Review comment:
       Yup makes sense, Have modified the logging statement and added a test case to validate
the same. 




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



Mime
View raw message