Skip to content

Commit f63f975

Browse files
committed
fix: remove product-config references from comments
1 parent 6e66a6d commit f63f975

2 files changed

Lines changed: 5 additions & 6 deletions

File tree

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

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
//! Builder for `hive-site.xml`.
22
//!
3-
//! Precedence (matches the pre-product-config-removal behaviour):
4-
//! 1. Defaults: `hive.metastore.port=9083` (required product-config property, default value).
3+
//! Precedence:
4+
//! 1. Defaults: `hive.metastore.port=9083`.
55
//! 2. Automatic / operator-injected: warehouse dir (hardcoded `/stackable/warehouse`),
66
//! metrics enabled (`true`), JDBC driver/url/credentials, S3, Kerberos, OPA.
77
//! 3. Spec: `warehouseDir` from the merged config overrides the hardcoded warehouse dir.
@@ -57,7 +57,7 @@ pub fn build(
5757
let database_connection_details = &cluster_config.metadata_database_connection_details;
5858
let mut data: BTreeMap<String, String> = BTreeMap::new();
5959

60-
// 1. Defaults (required product-config property `hive.metastore.port`).
60+
// 1. Defaults.
6161
data.insert(
6262
HIVE_METASTORE_PORT.to_string(),
6363
DEFAULT_HIVE_METASTORE_PORT.to_string(),

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

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,7 @@
11
//! Builder for `security.properties` (the JVM security properties file).
22
//!
3-
//! Reproduces the product-config behaviour: `networkaddress.cache.ttl` and
4-
//! `networkaddress.cache.negative.ttl` are `required` properties with recommended
5-
//! values, so they are always emitted; user `security.properties` overrides win.
3+
//! `networkaddress.cache.ttl` and `networkaddress.cache.negative.ttl` are `required`
4+
//! properties with recommended values, so they are always emitted.
65
76
use std::collections::BTreeMap;
87

0 commit comments

Comments
 (0)