Use Copy instead of Clone

Author: sbernauer

trino-lb-core/src/trino_cluster.rs

@@ -3,7 +3,7 @@ use std::{fmt::Display, time::SystemTime};
 use serde::{Deserialize, Serialize};
 use strum::IntoStaticStr;
 
-#[derive(Clone, Debug, Deserialize, Eq, PartialEq, Hash, Serialize, IntoStaticStr)]
+#[derive(Copy, Clone, Debug, Deserialize, Eq, PartialEq, Hash, Serialize, IntoStaticStr)]
 pub enum ClusterState {
     Unknown,
     /// Not running at all

trino-lb/src/http_server/admin/mod.rs

@@ -158,7 +158,7 @@ async fn set_cluster_activation(
 
     state
         .persistence
-        .set_cluster_state(cluster_name, desired_state.clone())
+        .set_cluster_state(cluster_name, desired_state)
         .await
         .context(SetClusterStateInPersistenceSnafu {
             cluster: cluster_name,

trino-lb/src/scaling/mod.rs

@@ -434,7 +434,7 @@ impl Scaler {
             let me = Arc::clone(&self);
             let target_state = target_states.get(&cluster.name).unwrap();
             join_set.spawn(
-                me.apply_cluster_target_state(cluster, target_state.clone())
+                me.apply_cluster_target_state(cluster, *target_state)
                     .instrument(Span::current()),
             );
         }