UGS displays total sync time incorrectly

Development Pipeline & Plugins
, ,
1

WorkspaceUpdate.cs has a time entry for total time, but when reporting total time it sums all the times. This double counts as the stop watch is active while the individual times are used. Given this is just for logging we’ve just not add the “Sync total” to the times list to remove the biggest offender.

times.Add(new Tuple<string, TimeSpan>("Sync Total", syncTelemetryStopwatch.Stop("Success")));
...

logger.LogInformation("Total time: {TotalTime}", FormatTime(times.Sum(x => (long)(x.Item2.TotalMilliseconds / 1000))));
2

Steps to Reproduce

  1. Sync via UGS
  2. Look at the log and see "Total time: " is roughly double the actual time.
3

Hi,

Thanks for reporting, I logged the issue as UE-362510 (ounce accepted to be visible publicly, it should appears at Unreal Engine Issues and Bug Tracker (UE\-362510\))é

Regards,

Patrick