ktmud commented on PR #19941: URL: https://github.com/apache/superset/pull/19941#issuecomment-1121702948
> No, I don't believe this was the intention. Then I don't understand why we have an [`end_dttm`](https://github.com/apache/superset/blob/6903c48c96fac555ca08fef3255c54522479f387/superset/models/reports.py#L204) (and "duration") for each log? If we are not updating the log record, i.e., it is a "fire and forgot" operation, shouldn't a `start_ddm` be enough? The log list table is also less readable when you have duplicate records with one having empty values and one having a spinner icon that never changes state. -- 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]
