bito-code-review[bot] commented on PR #35807:
URL: https://github.com/apache/superset/pull/35807#issuecomment-3434419948
<details open>
<summary><strong>Interaction Diagram by <a
href="https://bito.ai#sequence_diagram";>Bito</a></strong></summary>
```mermaid
sequenceDiagram
participant SCHED as Task Scheduler
participant CMD as AsyncExecuteReportScheduleCommand<br/>🔄 Updated | ●●○
Medium
participant STATE as BaseReportState<br/>🔄 Updated | ●●● High
participant HDR as HeaderDataType<br/>🔄 Updated | ●●○ Medium
participant EMAIL as EmailNotification<br/>🔄 Updated | ●●○ Medium
participant LOG as Logger
Note over STATE: Now includes execution_id<br/>in log data generation
SCHED->>CMD: execute(task_id, report_id, scheduled_time)
CMD->>STATE: create with execution_id
STATE->>HDR: _get_log_data() with execution_id
HDR-->>STATE: return enhanced log data
STATE->>EMAIL: send notification with header_data
EMAIL->>LOG: log with execution_id from header_data
LOG-->>EMAIL: log entry created
EMAIL-->>STATE: notification sent
STATE-->>CMD: execution complete
```
Critical path: Task
Scheduler->AsyncExecuteReportScheduleCommand->BaseReportState->HeaderDataType->EmailNotification->Logger
> **Note:** The report execution flow now includes execution_id tracking
throughout the notification pipeline. BaseReportState generates log data with
execution_id, which flows through HeaderDataType to EmailNotification for
enhanced logging visibility.
</details>
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
