lbp_config: remove needless string allocation

muzarski · muzarski · commit 7b8572a15f75 · 2024-10-24T12:47:15.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
@@ -65,9 +65,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
`@@ -65,9 +65,9 @@ impl LoadBalancingConfig {`
`65`	`65`	`builder =`
`66`	`66`	`builder.enable_shuffling_replicas(self.token_aware_shuffling_replicas_enabled);`
`67`	`67`	`}`
`68`		`- if let Some(dc_awareness) = self.dc_awareness.as_ref() {`
	`68`	`+ if let Some(dc_awareness) = self.dc_awareness {`
`69`	`69`	`builder = builder`
`70`		`- .prefer_datacenter(dc_awareness.local_dc.clone())`
	`70`	`+ .prefer_datacenter(dc_awareness.local_dc)`
`71`	`71`	`.permit_dc_failover(true)`
`72`	`72`	`}`
`73`	`73`	`if self.latency_awareness_enabled {`