move timing headers to send_serialized (#1379)

Summary:
Pull Request resolved: #1379

... because this is at the bottom of the send stack. Also move headers before message to be consistent.
ghstack-source-id: 313885566
exported-using-ghexport

Reviewed By: jayasi, shayne-fletcher

Differential Revision: D83570413

fbshipit-source-id: 329e266cf176a823e41cf37c8cf3bc69474dd486

Author: mariusae

hyperactor/src/reference.rs

@@ -1022,7 +1022,7 @@ impl<M: RemoteMessage> PortRef<M> {
                 MailboxSenderErrorKind::Serialize(err.into()),
             )
         })?;
-        self.send_serialized(cx, serialized, headers);
+        self.send_serialized(cx, headers, serialized);
         Ok(())
     }
 
@@ -1031,8 +1031,8 @@ impl<M: RemoteMessage> PortRef<M> {
     pub fn send_serialized(
         &self,
         cx: &impl context::Mailbox,
-        message: Serialized,
         mut headers: Attrs,
+        message: Serialized,
     ) {
         crate::mailbox::headers::set_send_timestamp(&mut headers);
         cx.post(self.port_id.clone(), headers, message);

monarch_simulator/src/controller.rs

@@ -113,7 +113,7 @@ impl ControllerMessageHandler for SimControllerActor {
         tracing::info!("controller send to ranks {:?}: {}", ranks, message);
         self.worker_actor_ref
             .port::<WorkerMessage>()
-            .send_serialized(cx, message, Attrs::new());
+            .send_serialized(cx, Attrs::new(), message);
         Ok(())
     }