jedcunningham commented on code in PR #30859:
URL: https://github.com/apache/airflow/pull/30859#discussion_r1187933332
##########
airflow/jobs/scheduler_job_runner.py:
##########
@@ -705,12 +705,14 @@ def _process_executor_events(self, session: Session) ->
int:
ti_primary_key_to_try_number_map[ti_key.primary] =
ti_key.try_number
self.log.info(
- "Executor reports execution of %s.%s run_id=%s exited with
status %s for try_number %s",
+ "Executor received event with state %s for task instance "
Review Comment:
```suggestion
"Executor sent event with state %s for task instance "
```
The executor is sending the event, scheduler job is receiving it.
##########
airflow/jobs/scheduler_job_runner.py:
##########
@@ -705,12 +705,14 @@ def _process_executor_events(self, session: Session) ->
int:
ti_primary_key_to_try_number_map[ti_key.primary] =
ti_key.try_number
self.log.info(
- "Executor reports execution of %s.%s run_id=%s exited with
status %s for try_number %s",
+ "Executor received event with state %s for task instance "
+ "dag_id=%s task_id=%s run_id=%s try_number=%s map_index=%s",
Review Comment:
I wonder if we should just toss ti_key here and let its str do it's thing.
Same info.
--
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]
