Skip to content

Commit 0e5d945

Browse files
committed
Trim redundant Rust comments in recorder/hindsight/exporter
Drop restate-only comments and banner dividers; keep format contracts and why-rationale. Comments-only; behavior unchanged.
1 parent 482f88b commit 0e5d945

4 files changed

Lines changed: 5 additions & 22 deletions

File tree

src/exporter/config/general.rs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,11 +3,9 @@ use crate::Url;
33

44
#[derive(Deserialize)]
55
pub struct General {
6-
// the exporter samples periodically, this controls that interval
76
#[serde(default = "interval")]
87
interval: String,
98

10-
// the listen address of the exporter
119
#[serde(default = "listen")]
1210
listen: String,
1311

src/hindsight/config/general.rs

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,19 +3,16 @@ use crate::Url;
33

44
#[derive(Deserialize)]
55
pub struct General {
6-
// how often to sample from the agent
76
#[serde(default = "interval")]
87
interval: String,
98

10-
// duration for the ringbuffer
119
#[serde(default = "duration")]
1210
duration: String,
1311

1412
// the address of the Rezolus agent
1513
#[serde(default = "source")]
1614
source: String,
1715

18-
// the path for output file
1916
#[serde(default = "output")]
2017
output: String,
2118

src/mcp/resource_usage.rs

Lines changed: 5 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -32,16 +32,13 @@ pub fn analyze_resource_usage(
3232
description: &str,
3333
top_n: usize,
3434
) -> Result<ResourceUsageResult, Box<dyn std::error::Error>> {
35-
// Get time range from TSDB
3635
let (start, end) = engine.get_time_range();
3736
let step = tsdb.interval();
38-
39-
// Execute the query
37+
4038
let result = engine.query_range(query, start, end, step)?;
41-
42-
// Extract samples from the result
39+
4340
use crate::viewer::promql::{QueryResult, Sample};
44-
41+
4542
let mut consumers = Vec::new();
4643
let mut total_sum = 0.0;
4744

@@ -62,7 +59,6 @@ pub fn analyze_resource_usage(
6259
let max = values.iter().cloned().fold(f64::NEG_INFINITY, f64::max);
6360
let min = values.iter().cloned().fold(f64::INFINITY, f64::min);
6461

65-
// Extract a readable name from labels
6662
let name = extract_consumer_name(&series.metric);
6763

6864
consumers.push(ResourceConsumer {
@@ -104,8 +100,7 @@ pub fn analyze_resource_usage(
104100
b.avg_usage.partial_cmp(&a.avg_usage)
105101
.unwrap_or(std::cmp::Ordering::Equal)
106102
});
107-
108-
// Calculate percentages and take top N
103+
109104
for consumer in &mut consumers {
110105
if total_sum > 0.0 {
111106
consumer.percent_of_total = (consumer.avg_usage / total_sum) * 100.0;
@@ -125,7 +120,7 @@ pub fn analyze_resource_usage(
125120

126121
/// Extract a readable name from metric labels
127122
fn extract_consumer_name(labels: &HashMap<String, String>) -> String {
128-
// Priority order for naming
123+
// Labels checked in priority order
129124
if let Some(name) = labels.get("name") {
130125
return name.clone();
131126
}
@@ -142,7 +137,6 @@ fn extract_consumer_name(labels: &HashMap<String, String>) -> String {
142137
return format!("id:{}", id);
143138
}
144139

145-
// Show the metric name if no good labels
146140
if let Some(metric_name) = labels.get("__name__") {
147141
return metric_name.clone();
148142
}
@@ -173,7 +167,6 @@ pub fn format_resource_usage(result: &ResourceUsageResult) -> String {
173167
result.total_usage
174168
));
175169

176-
// Header
177170
output.push_str(&format!(
178171
"{:<60} {:>10} {:>10} {:>10} {:>8}\n",
179172
"Consumer", "Avg", "Max", "Min", "% Total"
@@ -183,7 +176,6 @@ pub fn format_resource_usage(result: &ResourceUsageResult) -> String {
183176
"-".repeat(60), "-".repeat(10), "-".repeat(10), "-".repeat(10), "-".repeat(8)
184177
));
185178

186-
// Data rows
187179
for (i, consumer) in result.top_consumers.iter().enumerate() {
188180
output.push_str(&format!(
189181
"{:2}. {:<57} {:>10.2} {:>10.2} {:>10.2} {:>7.1}%\n",
@@ -196,7 +188,6 @@ pub fn format_resource_usage(result: &ResourceUsageResult) -> String {
196188
));
197189
}
198190

199-
// Show cumulative percentage
200191
let cumulative_percent: f64 = result.top_consumers.iter()
201192
.map(|c| c.percent_of_total)
202193
.sum();
@@ -207,7 +198,6 @@ pub fn format_resource_usage(result: &ResourceUsageResult) -> String {
207198
cumulative_percent
208199
));
209200

210-
// Add interpretation
211201
if result.top_consumers.len() > 0 {
212202
let top = &result.top_consumers[0];
213203
if top.percent_of_total > 50.0 {

src/mcp/server.rs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,6 @@ impl Server {
9999
}
100100
};
101101

102-
// Try to parse as JSON-RPC message
103102
let message: Value = match serde_json::from_str(&line) {
104103
Ok(msg) => msg,
105104
Err(e) => {
@@ -108,7 +107,6 @@ impl Server {
108107
}
109108
};
110109

111-
// Handle the message and get response
112110
if let Some(response) = self.handle_message(message).await? {
113111
let response_str = serde_json::to_string(&response)?;
114112
debug!("Sending response: {response_str}");

0 commit comments

Comments
 (0)