Skip to content

Commit ad32cef

Browse files
committed
used named tuple for readability
1 parent 53f7d59 commit ad32cef

File tree

1 file changed

+17
-14
lines changed

1 file changed

+17
-14
lines changed

rust/crd/src/history.rs

Lines changed: 17 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -292,12 +292,12 @@ impl SparkHistoryServer {
292292
);
293293

294294
// apply the role overrides
295-
let mut role_envs = role_env_overrides.iter().map(|env_var| {
295+
let mut role_envs = role_env_overrides.iter().map(|(env_name, env_value)| {
296296
(
297-
env_var.0.clone(),
297+
env_name.clone(),
298298
EnvVar {
299-
name: env_var.0.clone(),
300-
value: Some(env_var.1.to_owned()),
299+
name: env_name.clone(),
300+
value: Some(env_value.to_owned()),
301301
value_from: None,
302302
},
303303
)
@@ -306,16 +306,19 @@ impl SparkHistoryServer {
306306
vars.extend(&mut role_envs);
307307

308308
// apply the role-group overrides
309-
let mut role_group_envs = role_group_env_overrides.into_iter().map(|env_var| {
310-
(
311-
env_var.0.clone(),
312-
EnvVar {
313-
name: env_var.0.clone(),
314-
value: Some(env_var.1),
315-
value_from: None,
316-
},
317-
)
318-
});
309+
let mut role_group_envs =
310+
role_group_env_overrides
311+
.into_iter()
312+
.map(|(env_name, env_value)| {
313+
(
314+
env_name.clone(),
315+
EnvVar {
316+
name: env_name.clone(),
317+
value: Some(env_value),
318+
value_from: None,
319+
},
320+
)
321+
});
319322

320323
vars.extend(&mut role_group_envs);
321324

0 commit comments

Comments
 (0)