Fix a couple of nits with the timestamp additions.

TraceMachina · Jul 2, 2023 · b320de5 · b320de5
1 parent d30571e
commit b320de5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/cas/scheduler/action_messages.rs b/cas/scheduler/action_messages.rs
@@ -80,7 +80,7 @@ pub struct ActionInfo {
     pub platform_properties: PlatformProperties,
     /// The priority of the action. Higher value means it should execute faster.
     pub priority: i32,
-    /// When this action started to be loaded from the CAS
+    /// When this action started to be loaded from the CAS.
     pub load_timestamp: SystemTime,
     /// When this action was created.
     pub insert_timestamp: SystemTime,

diff --git a/cas/worker/running_actions_manager.rs b/cas/worker/running_actions_manager.rs
@@ -742,9 +742,9 @@ impl RunningAction for RunningActionImpl {
         output_folders.sort_unstable_by(|a, b| a.path.cmp(&b.path));
         output_file_symlinks.sort_unstable_by(|a, b| a.name_or_path.cmp(&b.name_or_path));
         output_directory_symlinks.sort_unstable_by(|a, b| a.name_or_path.cmp(&b.name_or_path));
-        execution_metadata.worker_completed_timestamp = (self.running_actions_manager.now_fn)();
         {
             let mut state = self.state.lock().await;
+            execution_metadata.worker_completed_timestamp = (self.running_actions_manager.now_fn)();
             state.action_result = Some(ActionResult {
                 output_files,
                 output_folders,