Skip to content

Commit 3879735

Browse files
committed
fix: use extend instead of loop + insert
1 parent 3c10d6b commit 3879735

2 files changed

Lines changed: 2 additions & 6 deletions

File tree

rust/operator-binary/src/controller/build/properties/hive_site.rs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -113,9 +113,7 @@ pub fn build(
113113
}
114114

115115
// 4. User overrides (highest precedence).
116-
for (k, v) in overrides {
117-
data.insert(k, v);
118-
}
116+
data.extend(overrides);
119117

120118
Ok(data)
121119
}

rust/operator-binary/src/controller/build/properties/security_properties.rs

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -31,9 +31,7 @@ pub fn build(overrides: BTreeMap<String, String>) -> BTreeMap<String, String> {
3131
// 2. No automatic operator-injected values.
3232
// 3. No merged_config contribution.
3333
// 4. User overrides (highest precedence).
34-
for (k, v) in overrides {
35-
props.insert(k, v);
36-
}
34+
props.extend(overrides);
3735

3836
props
3937
}

0 commit comments

Comments
 (0)