abhinavgautam01 commented on PR #1693: URL: https://github.com/apache/datafusion-ballista/pull/1693#issuecomment-4428896020
> maybe we should also set the `end_time` in case of job failure ? Good point — succeed_job already sets end_time, but fail_job never did, so FailedJob.ended_at could stay 0 while started_at was set. I’ve updated both StaticExecutionGraph and the AQE graph’s fail_job to set self.end_time to timestamp_millis() before building the failed status, mirroring success handling. The REST handlers still use saturating elapsed ms so inconsistent timestamps can’t panic if they ever appear again. -- 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]
