bito-code-review[bot] commented on PR #41177:
URL: https://github.com/apache/superset/pull/41177#issuecomment-4794621705
<!-- Bito Reply -->
The comment correctly identifies that referencing "master behavior" is
brittle, as it ties the explanation to a specific branch name that may change.
A more robust approach is to describe the behavior in terms of its functional
purpose or architectural role, such as "the default behavior" or "the existing
implementation."
**superset/commands/report/execute.py**
```
# Resolve the executor at the run() boundary the same way the
# default implementation does: tolerate a missing user
(find_user -> None)
# so the state machine still runs and its error envelope writes
the ERROR
# execution-log row and sends the owner notification.
```
--
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]
