lbp_config: remove needless string allocation

muzarski · muzarski · commit 3379c0d44b21 · 2024-10-17T13:25:55.000+02:00
`self` is consumed by this method, thus there is no need
to match dc_awareness by reference and to clone a local_dc string.
diff --git a/scylla-rust-wrapper/src/cluster.rs b/scylla-rust-wrapper/src/cluster.rs
@@ -55,9 +55,9 @@ impl LoadBalancingConfig {
             builder =
                 builder.enable_shuffling_replicas(self.token_aware_shuffling_replicas_enabled);
         }
-        if let Some(dc_awareness) = self.dc_awareness.as_ref() {
+        if let Some(dc_awareness) = self.dc_awareness {
             builder = builder
-                .prefer_datacenter(dc_awareness.local_dc.clone())
+                .prefer_datacenter(dc_awareness.local_dc)
                 .permit_dc_failover(true)
         }
         if self.latency_awareness_enabled {

Original file line number	Diff line number	Diff line change
`@@ -55,9 +55,9 @@ impl LoadBalancingConfig {`
`55`	`55`	`builder =`
`56`	`56`	`builder.enable_shuffling_replicas(self.token_aware_shuffling_replicas_enabled);`
`57`	`57`	`}`
`58`		`- if let Some(dc_awareness) = self.dc_awareness.as_ref() {`
	`58`	`+ if let Some(dc_awareness) = self.dc_awareness {`
`59`	`59`	`builder = builder`
`60`		`- .prefer_datacenter(dc_awareness.local_dc.clone())`
	`60`	`+ .prefer_datacenter(dc_awareness.local_dc)`
`61`	`61`	`.permit_dc_failover(true)`
`62`	`62`	`}`
`63`	`63`	`if self.latency_awareness_enabled {`