Skip to content

Commit 1f10a48

Browse files
committed
fix(doc): remove stale comments
1 parent ca9eb49 commit 1f10a48

4 files changed

Lines changed: 7 additions & 9 deletions

File tree

rust/operator-binary/src/controller/build/resource/config_map.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -173,7 +173,7 @@ pub fn build_rolegroup_config_map(
173173
data.insert(JVM_CONFIG.to_string(), jvm_config);
174174

175175
// 9. Vector agent config (`vector.yaml`) if the Vector agent is enabled. The file is templated
176-
// with environment variables injected by the v2 Vector container at runtime.
176+
// with environment variables injected by the Vector container at runtime.
177177
if rg.config.logging.enable_vector_agent {
178178
data.insert(
179179
VECTOR_CONFIG_FILE.to_string(),

rust/operator-binary/src/controller/build/resource/mod.rs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,7 @@
22
//!
33
//! Each submodule builds one kind of resource (ConfigMap, Service, Listener, PDB, …) and
44
//! *returns* it, rather than mutating shared state or applying it directly. The reconciler
5-
//! collects the returned objects and applies them. This mirrors the hive- and
6-
//! opensearch-operators' `controller/build/resource` layout.
5+
//! collects the returned objects and applies them.
76
87
pub mod config_map;
98
pub mod listener;

rust/operator-binary/src/controller/mod.rs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -71,8 +71,7 @@ pub struct ValidatedClusterConfig {
7171
/// A validated, merged Trino role-group config.
7272
///
7373
/// Holds the merged [`v1alpha1::TrinoConfig`] fields so the build steps consume this
74-
/// controller-owned type instead of the raw CRD struct (mirroring the opensearch- and
75-
/// hive-operators' `Validated…Config`).
74+
/// controller-owned type instead of the raw CRD struct.
7675
#[derive(Clone, Debug)]
7776
pub struct ValidatedTrinoConfig {
7877
pub affinity: StackableAffinity,
@@ -103,7 +102,7 @@ impl ValidatedTrinoConfig {
103102
/// Per-role configuration extracted during validation.
104103
///
105104
/// Lets the reconciler and build steps consume this controller-owned type instead of re-reading
106-
/// the raw [`v1alpha1::TrinoCluster`] (mirroring the hive-operator's `ValidatedRoleConfig`).
105+
/// the raw [`v1alpha1::TrinoCluster`].
107106
#[derive(Clone, Debug)]
108107
pub struct ValidatedRoleConfig {
109108
pub pdb: stackable_operator::commons::pdb::PdbConfig,

rust/operator-binary/src/controller/validate.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -109,9 +109,9 @@ pub type RoleGroupName = String;
109109

110110
/// Validated logging configuration for the Trino, prepare and (optional) Vector containers.
111111
///
112-
/// Produced up-front by [`validate_logging`] (mirroring the opensearch- and hive-operators) so
113-
/// that an invalid custom log ConfigMap name or a missing Vector aggregator discovery ConfigMap
114-
/// name fails reconciliation during validation rather than at resource-build time.
112+
/// Produced up-front by [`validate_logging`] so that an invalid custom log ConfigMap name or a
113+
/// missing Vector aggregator discovery ConfigMap name fails reconciliation during validation
114+
/// rather than at resource-build time.
115115
#[derive(Clone, Debug, PartialEq, Eq)]
116116
pub struct ValidatedLogging {
117117
pub prepare_container: ValidatedContainerLogConfigChoice,

0 commit comments

Comments
 (0)