bbovenzi commented on code in PR #71916:
URL: https://github.com/apache/airflow/pull/71916#discussion_r3833273865
##########
airflow-core/src/airflow/jobs/scheduler_job_runner.py:
##########
@@ -1320,7 +1316,10 @@ def _enqueue_executor_callbacks(self, session: Session)
-> None:
@staticmethod
def _process_task_event_logs(log_records: deque[Log], session: Session):
- objects = (log_records.popleft() for _ in range(len(log_records)))
+ objects = [log_records.popleft() for _ in range(len(log_records))]
+ for log_record in objects:
+ # A bulk insert skips the ORM hook that would stamp this from
``dag_id``.
+ log_record.team_name = resolve_team_name(log_record.dag_id,
session=session)
Review Comment:
For bulk log records, we should probably deduplicate how often we resolve
the team name. Could be hundreds of log_records but for only one or two dags.
--
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]
